-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Mono: Fix TLS on DSM7 #5070
Mono: Fix TLS on DSM7 #5070
Conversation
successfully installs and updates certificate store on DS-218+ with
|
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.
LGTM
don't forget to update SPK_REV
Thanks but how on application launch (e.g. sonarr) can we tell the mono where to find the new tls store? http://docs.go-mono.com/?link=man%3Amono(1) |
Alternatively every mono app needs to do this and store a copy because it's now using the home/user directory |
you shouldn't need to should you? the fix that has been working is it'd be nice to get this merged and pushed so it stops the deluge of support requests and thousands of users with broken NASes due to the bug. |
No unfortunately sudo is no longer available for packages in DSM7 On top of that the default path And needs root access so in this PR the path for DSM7 is changed to /var/packages/mono/var/.mono The 2nd part that is missing is to tell sonarr aka mono on launch about it hence why it is WIP. |
@@ -26,6 +26,7 @@ CONFIGURE_ARGS = --disable-mcs-build | |||
CONFIGURE_ARGS += --enable-btls | |||
CONFIGURE_ARGS += --disable-dependency-tracking | |||
CONFIGURE_ARGS += --without-mcs-docs | |||
CONFIGURE_ARGS += --prefix=$(INSTALL_PREFIX) |
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.
this is not required as already added with GNU_CONFIGURE = 1
CONFIGURE_ARGS += --prefix=$(INSTALL_PREFIX) |
Any chance to get this shipped soon? |
No idea, did you test if it works? I would need to test this on an untouched DSM or reinstall one because I think I modified my |
Ah sorry, I missed the fact that the CI uploads artifacts, I tested it and Sonarr still has the connection errors do I need an updated Sonarr package to test this as well? |
Just did a test and did not seem to work for me. My system setup: Steps exectuted:
observe PR package Sonarr logs example of 1 of the errors
/var/log/packages/mono.log (this log is only the PR package installation)
|
Thanks for the detailed log and information. Sorry that it doesn't work 😢 |
Can this be merged pls? :) |
hey @publicarray, as #5604 is now merged and published I believe this PR can be closed. |
Closing this PR as it has been superseded by #5604. |
Description
This should fix the trust store containing expired certificates in mono applications by using a different path. Dependent packages may need to be updated.
Fixes #5051
Checklist
all-supported
completed successfullyType of change