-
Notifications
You must be signed in to change notification settings - Fork 15
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
Dev duk ecodata adaptive changes #977
base: dev
Are you sure you want to change the base?
Conversation
@@ -25,6 +25,7 @@ class ApiKeyInterceptor { | |||
} | |||
|
|||
boolean before() { | |||
return true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we cannot include this
@@ -45,3 +45,4 @@ out | |||
/grails-app/assets/vendor/ | |||
/node_modules/ | |||
/grails-app/assets/dist/ | |||
/package-lock.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove
@@ -5,7 +5,7 @@ import org.apache.http.HttpStatus | |||
|
|||
import java.text.SimpleDateFormat; | |||
|
|||
@au.ala.org.ws.security.RequireApiKey(scopes=["ecodata/read"]) | |||
// @au.ala.org.ws.security.RequireApiKey(scopes=["ecodata/read"]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we cannot include this
properties.url = "sftp:https://upload.ala.org.au:" + dataProviderName + '/' + dataResourceId | ||
|
||
//properties.url = "sftp:https://upload.ala.org.au:" + dataProviderName + '/' + dataResourceId | ||
String url = grailsApplication.config.getProperty('ecodata.forCollectory.exportURL') ?: grailsApplication.config.getProperty('grails.serverURL')+"/ws/record/csvProject?id=" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically, we want to have it exactly as before i.e.
properties.url = grailsApplication.config.getProperty('ecodata.forCollectory.exportURL') + dataProviderName + '/' + dataResourceId
And application.groovy should have our default value "sftp:https://...". If you want a different url/code, then use a flag. And the default flag value should resolve to our url. The code should run as before without any configuration change. And you can make config change, to run it the way you want.
@@ -266,11 +266,13 @@ class PermissionService { | |||
Map rec=[:] | |||
rec.userId = it.userId | |||
rec.role = it.accessLevel?.toString() | |||
rec.displayName = "" // work around, when the following call to authService fails |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove. should be added in biocollect
out.put(it.userId,rec) | ||
|
||
} | ||
|
||
def userList = authService.getUserDetailsById(userIds) | ||
log.info("--> getMembersForProjectPerPage: userList for UserIds: "+userIds+":"+ userList ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
log.debug
@@ -1,11 +1,23 @@ | |||
{ | |||
"name": "ecodata", | |||
"version": "4.0", | |||
"version": "6.14.8", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does not correspond with ecodata version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cannot delete this
This PR contains 4 commits we'd like to share information about
Parameterize url for collectory access when using biocache ingest
(We will not use sftp: server)
workaround for displayName in PermissionsService
Reinhardt committed 2 weeks ago
adaptions for compiling locally
Reinhardt committed 2 weeks ago
Commits on Jun 10, 2024
security checks eliminated - for testing reasons only
Reinhardt committed last week
Commits on Jun 13, 2024
parameterize url for collectory access when using biocache ingest
wenzina188 committed last week