web-dev-qa-db-ja.com

非公開ノードの代替HTTPコードを返す

Drupal 8.の非公開ノードに対して、403応答ではなく404ページを返そうとしています。

私は kernel response subscriber をテストしましたが、使用しているコードはステータスコードを403から404に変更するだけで、実際には404ページを表示しませんでした。だから誰かが私に404ページのResponseオブジェクトを生成する方法を教えてもらえますか?

これは私が使っていたコードです:

class ResponseSubscriber implements EventSubscriberInterface {

  /**
   * {@inheritdoc}
   */
  public static function getSubscribedEvents() {
    return [KernelEvents::RESPONSE => [['alterResponse']]];
  }

  /**
   * Change status code to 404 from 403 if page is an unpublished node.
   *
   * @param FilterResponseEvent $event
   *   The route building event.
   */
  public function alterResponse(FilterResponseEvent $event) {
    if ($event->getResponse()->getStatusCode() == 403) {
      /** @var \Symfony\Component\HttpFoundation\Request $request */
      $request = $event->getRequest();
      $node = $request->attributes->get('node');
      if ($node instanceof Node && !$node->isPublished()) {
        $response = $event->getResponse();
        // This changes the code, but doesn't return a 404 page.
        $response->setStatusCode(404);

        $event->setResponse($response);
      }
    }
  }

}

私は最終的にこの応答サブスクライバーを完全に削除することに頼り、次のようにhook_node_accessを使用しました:

use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Drupal\Core\Access\AccessResult;

function unpublished_404_node_access(\Drupal\node\NodeInterface $node, $op, \Drupal\Core\Session\AccountInterface $account) {

  if ($op == 'view' && !$node->isPublished()) {
    if (\Drupal::moduleHandler()->moduleExists('workbench_moderation') && $account->hasPermission('view any unpublished content')) {
      return AccessResult::neutral();
    }
    elseif (\Drupal::routeMatch()->getRouteName() == 'entity.node.canonical' && \Drupal::routeMatch()->getRawParameter('node') == $node->id()) {
      throw new NotFoundHttpException();
      return AccessResult::neutral();
    }
  }

  return AccessResult::neutral();
}

これは、Drupal 7.の場合、このサイトのいくつかの回答と一致しているようです。しかし、hook_node_accessではなく、KernelEventサブスクライバーを使用してこれを行うより良い方法があるかどうかを確認したかったのです。ノードが403を返しているかどうかをテストしてから、404ページと404ステータスコードを含む新しい応答を生成したいのですが、その方法がわかりません。

8
oknate

これは、応答サブスクライバーの代わりに、例外の前半で試行できます。 HttpExceptionSubscriberBaseを拡張すると、これを行うために必要なコードが少なくなります。次に、メソッド$event->setException()を使用して、403を404例外に置き換えます

/ src/EventSubscriber/Unpublished404Subscriber.php

_<?php

namespace Drupal\mymodule\EventSubscriber;

use Drupal\Core\EventSubscriber\HttpExceptionSubscriberBase;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;

class Unpublished404Subscriber extends HttpExceptionSubscriberBase {

  protected static function getPriority() {
    // set priority higher than 50 if you want to log "page not found"
    return 0;
  }

  protected function getHandledFormats() {
    return ['html'];
  }

  public function on403(GetResponseForExceptionEvent $event) {
    $request = $event->getRequest();
    if ($request->attributes->get('_route') == 'entity.node.canonical') {
      $event->setException(new NotFoundHttpException());
    }
  }

}
_

mymodule.services.yml:

_services:
  mymodule.404:
    class: Drupal\mymodule\EventSubscriber\Unpublished404Subscriber
    arguments: []
    tags:
      - { name: event_subscriber }
_

これにより、正規ノードルートのすべての403例外が置き換えられます。これが本当にノードが非公開になっているためかどうかを確認したい場合は、ノードオブジェクト$request->attributes->get('node')を取得できます。

6
4k4