-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃殌 Feature: Omit the requirement of the URL parameter in server createMembership function #5675
Comments
Thank you for opening this. Currently, the URL parameter plays a crucial role in safeguarding against possible open redirect attacks, particularly when involving user API interactions. If we were to remove this parameter, it would require careful internal handling, as other adjustments would likely need to be made as well. |
I've meant with Server side SDKs, in which the |
Yes, I am talking about them as well |
Closing as url is no longer required |
馃敄 Feature description
From the docs
When using the teams
createMembership()
function in a Server side SDK it seems like there's no need for the URL parameter, or at least it shouldn't be required.馃帳 Pitch
As no email is being sent then no need for the
url
parameter.P.s. maybe it can work that if the user is adding
url
to the server side function, then it will send email even from the server side implementation. but that's already another feature.馃憖 Have you spent some time to check if this issue has been raised before?
馃彚 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: