Skip to content

Add a function to get the reserved url from the global variable.#3999

Merged
dgw merged 1 commit intoYOURLS:masterfrom
matt-h:fix/3769-missing-reserved-url
Oct 11, 2025
Merged

Add a function to get the reserved url from the global variable.#3999
dgw merged 1 commit intoYOURLS:masterfrom
matt-h:fix/3769-missing-reserved-url

Conversation

@matt-h
Copy link
Copy Markdown
Contributor

@matt-h matt-h commented Oct 11, 2025

This does proper checking to make sure that it is set and is an array. This prevents errors when it is used expecting to be an array if it wasn't set properly in the config.

Also added a test to confirm that the function still works if it was set to null

Fixes #3769

This does proper checking to make sure that it is set and is an array.
This prevents errors when it is used expecting to be an array if it wasn't set properly in the config.
Fixes YOURLS#3769
Copy link
Copy Markdown
Member

@ozh ozh left a comment

Choose a reason for hiding this comment

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

LGTM !

@dgw dgw merged commit 201789f into YOURLS:master Oct 11, 2025
6 checks passed
tomtenuta pushed a commit to tomtenuta/YOURLS that referenced this pull request Nov 4, 2025
…RLS#3999)

This does proper checking to make sure that it is set and is an array.
This prevents errors when it is used expecting to be an array if it wasn't set properly in the config.
Fixes YOURLS#3769
@dgw dgw mentioned this pull request Dec 17, 2025
4 tasks
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.

Missing $yourls_reserved_URL in config breaks adding shorturls

3 participants