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

Added experiment to execute client for user_groups #1810

Closed
wants to merge 1 commit into from

Conversation

Gabefire
Copy link
Collaborator

@Gabefire Gabefire commented Sep 12, 2024

Description

  • See the attached issue. The user is complaining about the endpoint that is used for user groups.
  • TLDR: client.execute has an experiment parameter outside the client. Setting the enable_experiment parameter swaps the endpoint used in the SDK from "/graphql" to "/_gql." From my understanding of these different URLs, they don't mean much anymore. The user must have some SSL certificate verification check in his local environment that monitors if the experiment is turned on for the correct URL because his other issue he has submitted is similar.
  • I just changed what is passed to the experiment parameter inside user_groups for consistency. I thought about slightly modifying client.execute to override the experiment parameter if enabled_experiment is its own, but not sure of downstream effects or the exact reason the two experiment places are different (maybe some legacy feature)

@Gabefire
Copy link
Collaborator Author

#1808

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.

User Group Management doesn't support experimental_enabled on Client
1 participant