Skip to content

Prevents unauthorized update to schedule#2466

Merged
pumfleet merged 1 commit intomainfrom
fix/unauthorized-schedule-update
Apr 12, 2022
Merged

Prevents unauthorized update to schedule#2466
pumfleet merged 1 commit intomainfrom
fix/unauthorized-schedule-update

Conversation

@hariombalhara
Copy link
Copy Markdown
Member

@hariombalhara hariombalhara commented Apr 12, 2022

What does this PR do?

As reported on slack, prevents unauthorized update to schedule

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Update the schedule and see that it's being updated as earlier
  • Update schedule, intercept request and edit schedule ID to some other user's schedule. Check that the request is rejected. Earlier it was accepted and other users' schedule was successfully updated.

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 12, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

docs – ./apps/docs

🔍 Inspect: https://vercel.com/cal/docs/A2Q9XXS4TEjCkCF5pTSxRYz2YQ6C
✅ Preview: Canceled

calendso – ./apps/web

🔍 Inspect: https://vercel.com/cal/calendso/FgWKh5rDLT3KbcTGjjpG8QFX7jzX
✅ Preview: https://calendso-git-fix-unauthorized-schedule-update-cal.vercel.app

@pumfleet pumfleet merged commit 31d1bde into main Apr 12, 2022
@pumfleet pumfleet deleted the fix/unauthorized-schedule-update branch April 12, 2022 09:22
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants