-
Notifications
You must be signed in to change notification settings - Fork 39
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
DOCUMENTATION????????????? #18
Comments
I agree that better documentation would be nice. To resolve your immediate problem: $cspBuilder->addDirective('report-uri', 'https://foo.example.com/csp_reporting.php'); |
I've added an explicit, dedicated API method and included it in the README, for the next minor release: c214593 |
I might write up a bit of documentation and send it your way. There are a few gotchas here and there. One followup question: How do I programmatically set "default-src" to 'none'? I've tried EDIT:
|
How do I best contribute improved documentation? Just fork and do a pull request on readme.md ? |
That would be the best way, but if you don't have the time to fiddle with it, just share a list of things you find frustrating, difficult, confusing, etc. and I'll make a checklist. |
How do you install via composer. Is there a line I can add to my main Composer file? |
Installing with Composer gives a message suggesting I install psr/http-message. You might want to add a bit in the docs saying why this is suggested. |
I've moved this to a dependency. PSR dependencies are low risk, and this makes static analysis easier. |
For the love of God people, could you please document this?
I've spent an hour trying to figure out how to set report-uri and simply nothing works. CSPBuilder.php on line 107 thinks "/csp_reporting.php" is an array. Or something. I don't know, because I have no $#%*)^% idea how it's intended to work. AddDirective()? AddSource()? SacrificeFirstBornAtMidnight()?
The text was updated successfully, but these errors were encountered: