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

Log wget errors. Also enable https. #184

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

painter1
Copy link
Contributor

Log wget errors. Previously it was impossible to tell why data was not coming.

A minor second improvement lets sdget.sh download https data; previously credentials problems prevented that. (Of course, only a brain-dead data node would serve CMIP* data by https only; but it happens.)

N.B. These changes are mainly in sdget.sh. Unfortunately I could not find a good place to put the file; see issue #165. I put it where it formerly was, in the same directory as sdget.py. In order to use this improvement, it would have to be copied into /etc/synda/sdt/bin (typically) or hidden in whatever tar file will be unpacked into that directory.

…t coming.

A minor second improvement lets sdget.sh download https data; previously
credentials problems prevented that.  (Of course, only a brain-dead data
node would serve CMIP* data by https only; but it happens.)
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

Successfully merging this pull request may close these issues.

1 participant