Skip to content

Conversation

@maksbotan
Copy link
Contributor

@maksbotan maksbotan commented Mar 27, 2025

Если в апишке есть несколько вложенных Permit, то в swagger попадают ограничения из самого внешнего, из-за этого вот такая апишка:

Permit '["user", "admin"] :>
  ( "view" :> Get '[JSON] Text
  :<|> Permit '["admin"] :> ("edit" :> Post '[JSON] Text)
  )

в сваггере сообщает, что у обеих ручек (view, edit) пермишены user, admin, хотя у второй должен быть только admin.

В этом PR это фиксится, теперь в swagger попадают ограничения из самого внутренного Permit.

Это согласовано с инстансом HasServer — самый глубокий хендлер обрабатывается последним, т.к. servant-server разбирает апишку снаружи внутрь.

CI для ghc 9.6, 9.8 падает, это к сожалению пока так и останется.

@maksbotan maksbotan force-pushed the maksbotan/fix-permit branch from 0a98401 to 74e5201 Compare March 27, 2025 20:54
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Это автосгенерированный файл для гитхабовского CI, можно не ревьюить.

tag: 16808faa4db0d3d94ff4006c5ca88691551ebe40
--sha256: E/VW3Xb7Nvefle2QkTckuWn8QB2YhbBCEBVO2y/xcbo=

source-repository-package
Copy link
Contributor Author

Choose a reason for hiding this comment

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

с момента прошлого релиза этого пакета наши фиксы в openid-connect попали в мастер

@abychkova
Copy link
Contributor

не хочешь пофиксить сборки для поздних версий компилятора? раз это не горит..

@maksbotan
Copy link
Contributor Author

@abychkova хорошо бы, но там проблема в том, что наши зависимости с 9.6/9.8 не собираются, а не сам web-template. И надо будет в них ПРы делать.

Конечно, когда-нибудь придётся...

@maksbotan maksbotan merged commit 9e62c2a into master Apr 5, 2025
4 of 6 checks passed
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