Fixes #1630: apoc.load.csv doesn't support basic auth #1726
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1630
To make the procedure behavior consistent with the other loaders,
I added the
apoc.load.csvParams
procedure,useful if I want to load a csv on a server that requires a header and / or a payload,
in the same way as
apoc.load.jsonParams
.One sentence summary of the change.
Proposed Changes (Mandatory)
A brief list of proposed changes in order to fix the issue:
http://user:password@host
apoc.load.csvParams
similarly toapoc.load.jsonParams
procedureLoadCsvTest.java
to cover cases of the related issueextractCredentialsIfNeeded(url, failOnError)
common method fromLoadJson.java
toUtil.java
LoadJsonTest.java
to cover same cases of the issuereaderFor(String fileName, Map<String, Object> headers, String payload)
method inFileUtils.java