Update to account if applications use single quotes instead of double quotes#5
Update to account if applications use single quotes instead of double quotes#5AlaricWhitney wants to merge 4 commits into
Conversation
Update to account if applications use single quotes instead of double quotes.
Added anti fragility checks
|
@peterhellberg Here's a PR for your consideration to improve your app! Thanks! |
fixed quote selection for multiple values in header
|
Not panicking seems like a good change. I think it may fix issue #6. Do you know if RFC 5988 permits the relation-type to be quoted with a single quote in addition to double quote? |
|
@dmitshur I do not think it would be considered to be quoted, but If my understanding of the EBNF is correct (I fully agree that this library shouldn't |
|
@AlaricWhitney I'm sorry that there has been no activity on this for a really long time. I have not been able to focus on open source work for quite some time. With the release of The resulting {"'next'":{"URI":"https://example.com/?page=2","Rel":"'next'","Extra":{}}}(Note that Example: https://go.dev/play/p/E5tQVHbNTsS |
Update to account if applications use single quotes instead of double quotes.
Example:
In its current state if you have rel with single quotes, the application panics. I have updated the regex to match single quotes, and I have updated the application to avoid panics.