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

Devel #273

Open
wants to merge 57 commits into
base: devel
Choose a base branch
from
Open

Devel #273

wants to merge 57 commits into from

Conversation

pkautio
Copy link
Contributor

@pkautio pkautio commented May 17, 2024

No description provided.

pkautio added 30 commits May 13, 2024 15:49
Update the code to use httr2 package instead of obsolete httr package.
Updated to use httr2 package instead of obsolete httr package.
Updated httr --> httr2 package
New version 2.1.1
Httr --> httr2
New version 2.1.1
Function to fetch Bearer token from Earthdata authentication server
Function to get Bearer Access token. Access token is used for all other accesses to Nasa website.
Updated MODIStsp to use httr2 package and Bearer token for the authentication.
@pkautio
Copy link
Contributor Author

pkautio commented May 17, 2024

  • Fix authentication issues by migrating from user authentication to bearer authentication.
  • Migrate from httr package to httr2 package

@LouiseLeroux
Copy link

Hi.
Thanks for this update of MODIStsp. I tried to install this updated version, but got that message or error :

Error in parse(outFile) : /tmp/Rtmp7CVJ1S/remotesba6acb1cae/MODIStsp-2.1.0/R/MODIStsp_download.R:47:35: unexpected '{' 46: # Fetch Bearer token to be used for further authentication 47: if (is.defined(earthdata_token)

Any idea?

@LouiseLeroux
Copy link

Hi.

Update : I have been able to install after after having corrected MODIStsp_download :

if (is.defined(earthdata_token)) { token <- earthdata_token } else { token <- get_earthdata_token(user, password) }

But still an issue when trying to download. It starts the process (creating the output folder) but stop on the following error :

[Mon May 27 13:33:06 2024] MODIStsp --> Starting processing Warning: Error in get_earthdata_token: object 'user' not found

@pkautio
Copy link
Contributor Author

pkautio commented May 27, 2024

How are you calling Modistsp`?

@LouiseLeroux
Copy link

LouiseLeroux commented May 27, 2024 via email

@maelle
Copy link
Member

maelle commented Sep 26, 2024

@pkautio any update? 😸

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.

4 participants