Skip to content
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

Multi-user shoutcast and removed corner images from web interface #242

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

TheTaLlesT
Copy link

@TheTaLlesT TheTaLlesT commented Dec 10, 2018

Removed corner images from web interface and replaced with CSS to keep original appearance.
Added hyphen, period, underscore, and tilde as safe URL characters (see rfc3986).
Adds the ability for shoutcast sources to define a login user and mount point by using the patterns below.

username:password@mount
username@mount:password
password@mount
@mount:password
username:password
password

If a mount point is not defined, it uses the configuration file defined mount.
If a user is not defined, it uses the default 'source' user.
This allows several shoutcast sources to log in at once using different mount points.
The flexibility in the login patterns is to optionally allow users to define the mountpoint in the non-obfuscated text boxes of their source client.

Hopefully my force push doesn't hurt anything. I'm still learning how to use git and I didn't know how to merge upstream changes. I think its right now.

karlheyes added a commit that referenced this pull request Jul 4, 2023
…242

Use the css aspects to use the in-built browser use which allows us to avoid
the extra requests to get the images.

The other part of the pull is the multi-user shoutcast compatible handling. That
already exists using a similar type of layout but whether there is something
different in the implementation will need to assessed.  The web page changes
though are fine.

karl.
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.

1 participant