Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

matt-h
Copy link
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
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
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