The previous package was outdated, preventing us from updating TS.
This one also lints YAML and JSON,
and applies many more rules to the test files,
explaining all the changes in this PR.
* patch: initial ideas around how to ignore invalid accept header parts. not fully tested, plenty of cleanup to do
* patch: finish removing commented code. finalize unit tests for headerUtil methods that now ignore bad header parts rather than throwing as before
* patch: remove @throws comments in HeaderUtil and update some comments to account for changes to returned values
* feat: Give option for header parsing to be strict
---------
Co-authored-by: Charlie Blevins <blevins.charlie@gmail.com>
Implemented new StringUtil helper functions: splitCommaSeparated, sanitizeUrlPart, isValidFileName.
Added helper functions to HeaderUtil: matchesAuthorizationScheme, hasScheme.
Added unit tests for the new helper functions.
Refactored codebase to use helper functions instead of regexes if applicable.
* feat: support storage and retrievel of content-type parameters
* test: extra unit tests for parseContentTypeWithParameters
* refactor: simplify set contentType()
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: simplify for loop because of unique blankNodes
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: ContentTypeParameter should be contentTypeParameter
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: remove undefined type in favor of var? syntax
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: use new parseContentType internally
* chore: remove commented code
* docs: code documentation line changed
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: Check for faulty metadata in contentType rdf structure
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: remove all instances of blanknodes
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: use full contentType when parsing header
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* refactor: use quads() method instead of store.getQuads()
* refactor: .value needed for type correctness
* feat: ReprMetadata constructor now supports full content-type string
Co-authored-by: Joachim Van Herwegen <joachimvh@gmail.com>
* fixes#995
* fixed the remarks and added a test for parseContentType
* refactor: remove unnecessary comment
Co-authored-by: woutslabbinck <wout.slabbinck@ugent.com>
* Fallback to X-Forwarded-* headers
This uses the first value from X-Forwarded-Host and
X-Forwarded-Proto if they're present and the standard Forwarded
header is not.
* Update parseForwarded to handle X-Forwarded-*
This updates the signature for parseForwarded to take in the headers
and handle the logic of falling back to X-Forwarded-* headers.
* Update src/util/HeaderUtil.ts
Co-authored-by: Ruben Verborgh <ruben@verborgh.org>
* Inline parseXForwarded helper
Additionally fixes a typo, updates a unit test, and removes a
typing that is no longer necessary.
* Tweak handling of X-Forwarded value checking and assignment
* Fix: terminology & consistency suggestions from review
Co-authored-by: Ruben Verborgh <ruben@verborgh.org>
Co-authored-by: Ruben Verborgh <ruben@verborgh.org>
Co-authored-by: Wouter Termont <woutermont@gmail.com>
* feat: Add logging
* refactor: Configure the logger for the tests once globally
* feat: Add logging
* fix: Fix ESLint errors due to merge conflicts
* Review log and error messages.
* refactor: Cleanup a bit
* refactor: Change to logger info calls
Co-authored-by: Ruben Verborgh <ruben@verborgh.org>
