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: any remaining open redirects #61

Merged
merged 4 commits into from
Mar 24, 2022
Merged

fix: any remaining open redirects #61

merged 4 commits into from
Mar 24, 2022

Conversation

stakach
Copy link
Member

@stakach stakach commented Mar 22, 2022

the continue param redirects internally or to whitelisted URLs
as per feedback from the recent pen testing

the continue param redirects internally or to whitelisted URLs
@stakach stakach added product: placeos Issue relates to PlaceOS Product focus: backend Focus on Back End Development priority: medium labels Mar 22, 2022
@stakach stakach requested review from viv-4, w-le and caspiano March 22, 2022 04:10
@stakach stakach self-assigned this Mar 22, 2022
@caspiano caspiano added the type: security Issue related to Security label Mar 22, 2022
@stakach stakach requested a review from caspiano March 22, 2022 23:54
@stakach stakach merged commit 607af54 into master Mar 24, 2022
@stakach stakach deleted the fix-open-redirect branch March 24, 2022 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus: backend Focus on Back End Development priority: medium product: placeos Issue relates to PlaceOS Product type: security Issue related to Security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants