Skip to content

отключить csrf для hapi20 getModuleResources#533

Closed
VladislavNsk wants to merge 1 commit into
masterfrom
fix/hapi20-get-module-resources-csrf
Closed

отключить csrf для hapi20 getModuleResources#533
VladislavNsk wants to merge 1 commit into
masterfrom
fix/hapi20-get-module-resources-csrf

Conversation

@VladislavNsk

@VladislavNsk VladislavNsk commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

В hapi16 get module resource выставлял флаг crumb ignore = true
в hapi20 get module resource видимо забыли

Из-за чего сейчас загрузка модулей падает с 403 ошибкой и приходится прокидывать вторым аргументом этот флаг в лоадер

plugins: {
      crumb: {
          ignore: true,
      },
  },

https://git.moscow.alfaintra.net/projects/CORP-GOODS-PURCHASES/repos/corp-buyers-ui/pull-requests/35/diff#src%2Fserver%2Fplugins%2Fget-module-resources.ts?t=32

@changeset-bot

changeset-bot Bot commented Jul 9, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 4d62ff5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@alfalab/scripts-server Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@qrik116

qrik116 commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

...routeParams,
plugins: {
...routePlugins,
crumb: {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть подозрения, что эта настройка не должна относиться к arui-scripts, так как arui-scripts ничего не знает про плагин crumb, к тому же он не идёт по умолчанию вместе с hapi. Или я что-то путаю?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Запутался в пакетах

я имел ввиду про 16 версию репы corp-front
https://git.moscow.alfaintra.net/projects/COR/repos/corp-front/browse/packages/hapi-16-get-module-resources/src/get-module-resources.ts#47

При переходе на аруи скриптс модули словили 403 ошибку

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом если считаете, что это не задача аруи скриптс, могу закрыть ПР, так как блокера нету и мы сейчас на проекте сами выставляем этот флаг (в описании ПРа прикладывал ссылку на пример)

Просто показалось что это каснется всех и можно дефолт тут выставить

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не смотря на то, что пакет внутренний носит название hapi-16, внутри он работает вовсе не на 16. К сожалению он не попал под переименования и остался с текущим названием. Если вы вдруг использовали раньше его, а теперь просто обновляетесь на новые версии hapi - можно просто обновить внутренний пакет и продолжить пользоваться им. Внешний вам скорее всего не нужен.

@heymdall-legal

Copy link
Copy Markdown
Collaborator

Данная доработка если и нужна, то точно не в этом пакете. Это внутренние детали реализации модулей и серверов в конкретном нарпавлении.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants