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

fix: bypass IP restriction for the methods required for our socketio backend (backport #26624) #26627

Merged

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented May 31, 2024

Reference: support ticket 15683

Socketio backend needs some information from the user, exposed via these endpoints:

  • Permission check
  • Basic user information

For now this seems to be the best way, maybe in the future if we rewrite the socketio backend in python we can have this be a direct method call.


This is an automatic backport of pull request #26624 done by Mergify.

…backend

Those requests are made from a separate backend, not by the user.

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
(cherry picked from commit 68fd322)
@mergify mergify bot requested review from a team and ankush and removed request for a team May 31, 2024 11:14
@akhilnarang akhilnarang merged commit b193396 into version-14-hotfix May 31, 2024
10 checks passed
@akhilnarang akhilnarang deleted the mergify/bp/version-14-hotfix/pr-26624 branch May 31, 2024 11:31
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 14.76.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants