search

rspace-os / rspace-web

This the main repository for RSpace: rspace-web. Check https://github.com/rspace-os/rspace-web/blob/main/DevDocs/DeveloperNotes/GettingStarted/GettingStarted.md for setup instructions.
https://github.com/rspace-os/rspace-web/wiki
GNU Affero General Public License v3.0
12 stars 3 forks source link

ServiceLoggerAspct logs unnecessary info #42

Closed ll4strw closed 2 months ago

ll4strw commented 2 months ago

Dear RSpace Team,

when logging in to an external disk via https://github.com/rspace-os/rspace-web/blob/1de072f022fa43c7e7d22098640149ee5068fc4d/src/main/java/com/researchspace/service/NfsManager.java#L53-L59

then the following logging function

https://github.com/rspace-os/rspace-web/blob/1de072f022fa43c7e7d22098640149ee5068fc4d/src/main/java/com/researchspace/webapp/controller/ServiceLoggerAspct.java#L103-L105

results in, for instance, nfs credentials to be logged unnecessarily.

mKowalski256 commented 2 months ago

Thank you for raising the issue, the logging should be fixed now with commit 228b05c.