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

Update directives to add manifest-src to Chrome/Firefox #339

Merged
merged 4 commits into from
Jul 14, 2017

Conversation

carlosantoniodasilva
Copy link
Contributor

@carlosantoniodasilva carlosantoniodasilva commented Jul 12, 2017

Closes #338.

  • Adds the manifest-src CSP directive to Chrome/Firefox
  • Removes reflected-xss

I also took the opportunity to bump the ruby version to the latest. I did not update the Changelog, let me know if that's necessary and I will.

If you have any questions/considerations, let me know, thanks!

@oreoshake oreoshake changed the base branch from master to 3.x July 13, 2017 23:54
@oreoshake oreoshake changed the base branch from 3.x to master July 13, 2017 23:54
@oreoshake
Copy link
Contributor

Yay! Thanks @carlosantoniodasilva. I can update the changelog.

Since master represents the (unreleased) 4.x line, I'll cherry-pick these into 3.x and push a new gem next week.

@oreoshake
Copy link
Contributor

(leaving this open so I remember to do this next week)

@carlosantoniodasilva
Copy link
Contributor Author

@oreoshake thanks! I totally missed the 3.x branch vs master being 4x. Let me know if you'd prefer a separate PR targeting 3.x or if I can help with anything else!

@oreoshake
Copy link
Contributor

oreoshake commented Jul 14, 2017 via email

@oreoshake oreoshake merged commit fed16c6 into github:master Jul 14, 2017
@oreoshake
Copy link
Contributor

Released in 3.6.7! Thanks again.

@carlosantoniodasilva
Copy link
Contributor Author

🎉 thanks so much @oreoshake! And sorry to hear about the 🔪 accident!

@carlosantoniodasilva carlosantoniodasilva deleted the manifest-src branch July 15, 2017 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants