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

Harvest: OAI client should follow http->https redirect since oai server validation allows http. #4831

Closed
kcondon opened this issue Jul 10, 2018 · 5 comments

Comments

@kcondon
Copy link
Contributor

kcondon commented Jul 10, 2018

Configuring an oai client allows http to be used in oai server url, as long as the server is valid. However, the actual harvest fails when it receives a 302 to move to https.

@cmbz
Copy link

cmbz commented Aug 20, 2024

To focus on the most important features and bugs, we are closing issues created before 2020 (version 5.0) that are not new feature requests with the label 'Type: Feature'.

If you created this issue and you feel the team should revisit this decision, please reopen the issue and leave a comment.

@cmbz cmbz closed this as completed Aug 20, 2024
@stevenferey
Copy link
Contributor

Hello,

I tried to harvest an OAI server compatible with the HTTP protocol, e.g.: http://cds.unistra.fr/registry/?verb=ListIdentifiers&metadataPrefix=oai_dc

Tested with the devleop branch, it is not possible to fill in an OAI server with the HTTP protocol :

Capture d’écran du 2024-09-26 18-13-43

It is however possible to declare a harvesting client with the API and fill in an OAI server with the HTTP protocol.
In this case, an error is raised during harvesting :

Caused By Exception.................... java.nio.channels.UnresolvedAddressException null
StackTrace: 
java.base/sun.nio.ch.Net.checkAddress(Unknown Source)
java.base/sun.nio.ch.Net.checkAddress(Unknown Source)
java.base/sun.nio.ch.SocketChannelImpl.checkRemote(Unknown Source)

@pdurbin
Copy link
Member

pdurbin commented Sep 30, 2024

@stevenferey interesting. Are you considering creating a pull request to fix this? 😄

@stevenferey
Copy link
Contributor

This is not planned by the team at the moment !

Do you want a new issue to be created to allow OAI server harvesting with HTTP protocol?

@pdurbin
Copy link
Member

pdurbin commented Oct 1, 2024

Sure, if you're interested in this, I think a fresh issue would be nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🔍 Interest
Development

No branches or pull requests

4 participants