Skip to content

slight improvement to Rack::Protection::ReferrerPolicy docs#2099

Open
davetron5000 wants to merge 1 commit intosinatra:mainfrom
davetron5000:improve-referrer-policy-docs
Open

slight improvement to Rack::Protection::ReferrerPolicy docs#2099
davetron5000 wants to merge 1 commit intosinatra:mainfrom
davetron5000:improve-referrer-policy-docs

Conversation

@davetron5000
Copy link

Problem

Rack::Protection::ReferrerPolicy are generally decent, but not formatted like the other changes I have been making

Solution

  • Set header as in other classes
  • Slightly expand upon the issue this solves
  • More explicitly document the options and behavior

# Supported browsers:: mixed support
# More infos:: https://www.w3.org/TR/referrer-policy/
# https://caniuse.com/#search=referrer-policy
# Supported browsers:: Most modern browsers from 2018 onwards. See https://caniuse.com/#search=referrer-policy for
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing the inclusion of this in most of these classes predates widespread use of caniuse.com, so I've been trying to sorta summarize and send a direct link. I dunno, maybe these should be removed entirely?

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.

1 participant