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

Update some FDSN WS urls #2169

Merged
merged 3 commits into from Jun 15, 2018

Conversation

Projects
None yet
2 participants
@claudiodsf
Member

claudiodsf commented Jun 7, 2018

IPGP has migrated its EIDA node to RESIF, it however still maintains an FDSN WS.
The current address is still http://eida.ipgp.fr, but it will soon change to http://fdsnws.ipgp.fr.

This pull request is a reminder of the fact that the URL mapping for IPGP should be changed as soon as the URL changes.

I hope that IPGP will be able to change the URL before the next maintenance release of ObsPy.

I also updated the link in the comments pointing to EIDA node list.

@claudiodsf claudiodsf added this to the 1.1.1 milestone Jun 7, 2018

@megies megies added the .clients.fdsn label Jun 7, 2018

@claudiodsf

This comment has been minimized.

Member

claudiodsf commented Jun 14, 2018

The new URL is live (http://fdsnws.ipgp.fr) and I pushed the change of the alias.

I also changed the arclink url in the arclink.Client() documentation to arclink.ipgp.fr:18001.

This is good to merge from my point of view.

@@ -46,7 +46,7 @@ class FDSNNoDataException(FDSNException):
"GFZ": "http://geofon.gfz-potsdam.de",
"ICGC": "http://ws.icgc.cat",
"INGV": "http://webservices.ingv.it",
"IPGP": "http://eida.ipgp.fr",
"IPGP": "http://fdsnws.ipgp.fr",

This comment has been minimized.

@megies

megies Jun 14, 2018

Member

@claudiodsf can you please also adapt the doctest in __init__.py, thanks

This comment has been minimized.

@claudiodsf

claudiodsf Jun 14, 2018

Member

Done. I updated the commit and force-pushed.

@megies

This comment has been minimized.

Member

megies commented Jun 14, 2018

+TESTS:clients.fdsn

@claudiodsf

This comment has been minimized.

Member

claudiodsf commented Jun 14, 2018

Done. Squashed into the previous commit and force-pushed.

@megies

megies approved these changes Jun 15, 2018

Thanks for the update 🚀

@megies megies merged commit 1b771a6 into obspy:maintenance_1.1.x Jun 15, 2018

2 of 4 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
docker-testbot docker testbot results not available yet
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@megies megies deleted the claudiodsf:fdsn_urls branch Jun 15, 2018

@claudiodsf

This comment has been minimized.

Member

claudiodsf commented Jun 15, 2018

Thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment