Skip to content

fix synology#503

Open
tiquis0290 wants to merge 7 commits intomauriceboe:devfrom
tiquis0290:dev
Open

fix synology#503
tiquis0290 wants to merge 7 commits intomauriceboe:devfrom
tiquis0290:dev

Conversation

@tiquis0290
Copy link
Copy Markdown
Contributor

@tiquis0290 tiquis0290 commented Apr 7, 2026

Changing how synology threats endpoint

Changing auth method in synology from v3 to v6 and adding OTP (MFA) support

@tiquis0290
Copy link
Copy Markdown
Contributor Author

because of pr i made to get the changes to the branch befor it got siconected hopefully now its all as required

Copy link
Copy Markdown
Collaborator

@jubnl jubnl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since you normalize the urls in the DB, could you please add this normalization part when you save the base url ? or just add a warning in the frontend saying something along the line of "Are you sure this normal that this url does not end with /photo"
And in the user settings, please add a bit of documentation regarding sonology (sonology server requirements, so DSM version and so on)

@tiquis0290
Copy link
Copy Markdown
Contributor Author

i normalize url each time iam calling it, i need it to append /photo to old urls in db

so dont see necessary to use it when storing new ones

sorry i missed the console log

@tiquis0290 tiquis0290 requested a review from jubnl April 7, 2026 22:49
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.

2 participants