-
Notifications
You must be signed in to change notification settings - Fork 14
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
DDFBRA-138 - Create task for fetching UUID of the graphql_consumer #1726
DDFBRA-138 - Create task for fetching UUID of the graphql_consumer #1726
Conversation
UUID is used while calling the GraphQL API.
1a49f4f
to
aa1b405
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should make the Drupal part as a drush cmd
logic for fetching consumer UUID from the .module file and into the new Commands class. Updated the Taskfile.yml to use the new drush command instead of calling a function using php-eval.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, but I think you should use 15 min in changing: dpl_consumers_get_consumer_uuid ()
so it throws an error if you cannot resolve the uuid
to be the correct one. Also now throws exception whenever we do not get the expected results returned.
Link to issue
https://reload.atlassian.net/browse/DDFBRA-138
Description
Added a new task for fetching the UUID of the graphql_consumer. This UUID is used while calling the GraphQL API.
The task is:
task dev:dpl-go:get-graphql-credentials
It fetches the UUID which should be set in the request headers as X-Consumer-ID