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

ETQ Intégrateur API, je voudrai que le timeout de l'API soit plus long #9380

Merged
merged 2 commits into from
Aug 2, 2023

Conversation

tchak
Copy link
Member

@tchak tchak commented Aug 2, 2023

No description provided.

@tchak tchak changed the title RTQ Intégrateur API, je voudrai que le timeout de l'API soit plus long ETQ Intégrateur API, je voudrai que le timeout de l'API soit plus long Aug 2, 2023
@what-the-diff
Copy link

what-the-diff bot commented Aug 2, 2023

PR Summary

  • Augmentation of query_info method in app/graphql/api/v2/context.rb
    A new item, graphql_mutation, has been added to the query_info method. This will provide more comprehensive details regarding the specific GraphQL mutation undertaken.

  • Increase of timeout limit in app/graphql/api/v2/schema.rb
    The max_seconds parameter for timeout has been increased from 10 to 30. This change reflects an adjustment in the time that the system will wait before aborting a process, hence providing additional time for complex database operations to finish.

  • Enhancement to event logging in config/initializers/lograge.rb
    A graphql_mutation key has been added to the event.payload hash. This helps in achieving better visibility into event logs by highlighting GraphQL mutation details.

@tchak tchak force-pushed the graphql-timeout branch from d743dac to b404064 Compare August 2, 2023 11:21
@tchak tchak enabled auto-merge August 2, 2023 11:22
@tchak tchak added this pull request to the merge queue Aug 2, 2023
Merged via the queue into demarches-simplifiees:main with commit 0bed8bd Aug 2, 2023
@tchak tchak deleted the graphql-timeout branch August 2, 2023 11:46
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.

2 participants