Skip to content
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

Urlencode resuptionToken #23

Closed
justinkelly opened this issue Sep 27, 2021 · 2 comments
Closed

Urlencode resuptionToken #23

justinkelly opened this issue Sep 27, 2021 · 2 comments

Comments

@justinkelly
Copy link
Contributor

Firstly, thanks for the amazing project!!

  • it's working 99% perfect to do full oai harvests

1 issue I have is that 1 of the OAI feeds includes un-escaped characters in the resumptionToken

  • refer below for examples

Unescaped + - this url doesn't work

https://rosetta.slv.vic.gov.au/oaiprovider/request?resumptionToken=20210901000000@20210926235959@Primo@oai_dc@AAAWxpABYAAALK+AAU&verb=ListRecords

Escaped + - this url does work

https://rosetta.slv.vic.gov.au/oaiprovider/request?resumptionToken=20210901000000@20210926235959@Primo@oai_dc@AAAWxpABYAAALK%2BAAU&verb=ListRecords

Is there a way to escape the resumptionToken with metha ?

  • or a config options

Cheers

Justin Kelly

justinkelly added a commit to biblio-ai/metha that referenced this issue Sep 27, 2021
justinkelly added a commit to biblio-ai/metha that referenced this issue Sep 27, 2021
justinkelly added a commit to biblio-ai/metha that referenced this issue Sep 27, 2021
justinkelly added a commit to biblio-ai/metha that referenced this issue Sep 27, 2021
Encode + in resumptionToken - miku#23

Encode + in resumptionToken - miku#23
@justinkelly
Copy link
Contributor Author

patch to fix this problem has been submitted in pull request #24

miku pushed a commit that referenced this issue Sep 27, 2021
miku pushed a commit that referenced this issue Sep 27, 2021
miku pushed a commit that referenced this issue Sep 27, 2021
@miku
Copy link
Owner

miku commented Sep 27, 2021

Thanks for the report - and fix!

@miku miku closed this as completed Sep 27, 2021
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

No branches or pull requests

2 participants