-
Notifications
You must be signed in to change notification settings - Fork 28
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
User + key management #17
Comments
Actually, this was implemented at some point. Then I decided to remove it, for simplicity, as both the webapi and CDS use unique UID schemes (ECWMF uses email, CDS a random number). A week ago I tried to implement the CAMS atmospheric data and their api (http:https://macc-raq-op.meteo.fr/doc/Guide_Numerical_Data_CAMS_new.pdf). Obviously, this one uses the email address again. 🤦♂️ So, yes... it would make sense to future proof things. |
Also, don't mind the issues, as long as you help out solving them 😄 |
Perfect! |
I think this should be documented well, then this shouldn't be too much of an issue. |
Actually, if you are still working on the readLines approach this could be a question that pops up. So people don't have to punch in the argument either. But I guess you had that covered. |
Yes, that's when it came up. |
In thinking about the user verification it occur to me that currently there's no way of knowing is an user/password belongs to the webapi or the cds service. Should there be a
service
argument towf_set_key()
and the rest?PS: You are free to tell me if I'm opening too many issues 😅
The text was updated successfully, but these errors were encountered: