pagure

#4506 More CSP headers fix
Merged by pingou. Opened by pingou.
more_csp  into  master

Download 4506.patch

All the commits in this PR can be reviewed separately :)

bf0bee4 :thumbsup:
b5b97e7 :thumbsup:
e007551 :thumbsup:
4ddc5d7:

pagure/templates/add_deploykey.html should not cancel_btn() be cancel_btn? :smile:
the rest seems ok

d74a3322: don't see any relation between this and csp fixes, otherwise :thumbsup:

pagure/templates/add_deploykey.html should not cancel_btn() be cancel_btn? 😄 the rest seems ok

Good catch, thanks :)

5 new commits added

  • Catch ImportError before trying to catch any fedora_messaging exceptions
  • Drop all the inline "onclick()" calls
  • Fix typo in the id name: publicnotifications != publicnotificationsb
  • Fix the id vs class in the repo_stats template
  • Move the wait_for_task function down and fix the call to setTimeout

5 new commits added

  • Catch ImportError before trying to catch any fedora_messaging exceptions
  • Drop all the inline "onclick()" calls
  • Fix typo in the id name: publicnotifications != publicnotificationsb
  • Fix the id vs class in the repo_stats template
  • Move the wait_for_task function down and fix the call to setTimeout

Looks fine now :thumbsup:

Jenkins is happy, let's get this in :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.