Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Raise error from helsinki profile endpoint if Keycloack token has expired #1465

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

matti-lamppu
Copy link
Collaborator

@matti-lamppu matti-lamppu commented Dec 11, 2024

🛠️ Changelog

  • See title.
  • Also allow fetching user by user_id. Requires can_view_user permission.

🧪 Test plan

  • Automated tests

🚧 Dependencies

  • None

🎫 Tickets

@matti-lamppu matti-lamppu added the feature Adds a new feature label Dec 11, 2024
@matti-lamppu matti-lamppu self-assigned this Dec 11, 2024
@matti-lamppu matti-lamppu force-pushed the helsinki-profile-endpoint-error branch from c36a951 to 04f5331 Compare December 16, 2024 11:04
@matti-lamppu matti-lamppu marked this pull request as ready for review December 16, 2024 11:13
@matti-lamppu matti-lamppu force-pushed the helsinki-profile-endpoint-error branch from 04f5331 to 18db215 Compare December 16, 2024 12:29
tilavarauspalvelu/api/graphql/schema.py Outdated Show resolved Hide resolved
tilavarauspalvelu/api/graphql/schema.py Outdated Show resolved Hide resolved
@matti-lamppu matti-lamppu force-pushed the helsinki-profile-endpoint-error branch from 18db215 to 75ac386 Compare December 17, 2024 07:39
@matti-lamppu matti-lamppu force-pushed the helsinki-profile-endpoint-error branch from 75ac386 to 6b69e7f Compare December 17, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adds a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants