Skip to content

Add material package restrictions #6996

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.

Sign up for GitHub

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

Jump to bottom
Merged
ThiefMaster merged 6 commits into from
Jul 30, 2025
Merged

Add material package restrictions #6996

ThiefMaster merged 6 commits into from
Jul 30, 2025

Conversation

@ThiefMaster
Copy link
Member

@ThiefMaster ThiefMaster commented Jul 29, 2025 •
edited
Loading

  • Add CAPTCHA to the user-facing material package form (only if not logged in)
  • Add rate limit for material packages generated by regular users
  • Add setting to restrict material packages (default to event managers)

@ThiefMaster ThiefMaster added this to the v3.3 milestone Jul 29, 2025
@ThiefMaster ThiefMaster changed the title Add CAPTCHA to material package form Add CAPTCHA + rate limit to material package form Jul 29, 2025
@ThiefMaster ThiefMaster marked this pull request as ready for review July 29, 2025 15:24
@ThiefMaster ThiefMaster added the build-wheel Build a Python wheel for this PR label Jul 29, 2025
@ThiefMaster ThiefMaster changed the title Add CAPTCHA + rate limit to material package form Add material package restrictions Jul 29, 2025
@tomasr8 tomasr8 self-requested a review July 30, 2025 09:28
tomasr8
tomasr8 reviewed Jul 30, 2025
View reviewed changes
@ThiefMaster
Add test
84a1a7a
@ThiefMaster
Disable captcha for authenticated users
92e92ef 
We already have the rate limit anyway
@ThiefMaster ThiefMaster merged commit 1a170c5 into indico:master Jul 30, 2025
11 checks passed
@ThiefMaster ThiefMaster deleted the material-pkg branch July 30, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tomasr8 tomasr8 tomasr8 approved these changes

Assignees

No one assigned

Labels

build-wheel Build a Python wheel for this PR

Projects

None yet

Milestone

v3.3

Development

Successfully merging this pull request may close these issues.

2 participants

@ThiefMaster @tomasr8