-
-
Notifications
You must be signed in to change notification settings - Fork 489
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
Service context null pointer #7593
Conversation
…ed in GeoNetwork and requires authentication to access it. Fixes geonetwork#7540 (geonetwork#7556)
The following also use the same context code. So they will probably have the same issue. So they should be fixed as well. /datastorages/jcloud/src/main/java/org/fao/geonet/api/records/attachments/JCloudStore.java |
I fixed most of them. But here is some leftover which I am not capable to figure out
They are about context.getUserSession(); but context here is null. I am sure it will cause null pointer |
* Service context null pointer * Service context null pointer for JCloud and S3
* Service context null pointer * Service context null pointer for JCloud and S3
Based on this PR #7556 there seems to be some bug in cmisStore where the ServiceContext was passed as null object.
The issue is not reproduceable in the 4.0 main branch due to the mechanism difference. But it's reproduceable in the 3.12 backport commit ianwallen@91795de
The null pointer avoidance is needed in order to make it work
Checklist
main
branch, backports managed with labelREADME.md
filespom.xml
dependency management. Update build documentation with intended library use and library tutorials or documentation