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

Various little fixes. #927

Closed
wants to merge 3 commits into from
Closed

Various little fixes. #927

wants to merge 3 commits into from

Conversation

mpmc
Copy link
Contributor

@mpmc mpmc commented Feb 16, 2017

I noticed that the "Basic" streaming and DVR options weren't set when creating users via the wizard setup. This commit adds those options as default.

Edit: Additional fixes.. I noticed some browsers will try and login using cached details sometimes causing a login loop when clicking the "Default Login" link, replacing it with /logout should hopefully prevent this.

@mpmc mpmc force-pushed the fix branch 3 times, most recently from 9281849 to cc86d4d Compare March 2, 2017 09:17
@mpmc mpmc force-pushed the fix branch 9 times, most recently from b48c1e3 to 296b949 Compare March 14, 2017 14:22
@mpmc mpmc force-pushed the fix branch 3 times, most recently from 025e12f to 81d567e Compare March 18, 2017 11:27
@mpmc mpmc changed the title Add missing default access entry rights Various little fixes. Mar 27, 2017
@perexg
Copy link
Contributor

perexg commented Mar 27, 2017

The permissions fix was not correct. I tried to fix this in b86ac23 . Applied the other two commits. Merged.

@perexg perexg closed this Mar 27, 2017
@mpmc
Copy link
Contributor Author

mpmc commented Mar 27, 2017

I had a feeling it wasn't right, but it seemed to work when I tried it, but hey-ho I'm no C developer (and it shows! 🤣 ). I really appreciate you fixing it and letting me know that I goofed!

@mpmc mpmc deleted the fix branch March 27, 2017 16:00
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.

None yet

2 participants