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

Wizard transforms basic auth credentials to lowercase when entered as part of portal URL #591

Closed
jmzagorski opened this issue Feb 12, 2018 · 4 comments
Labels

Comments

@jmzagorski
Copy link

Platform & OS Version
Android 7.0

The version of the app you are reporting:
1.2.515

Device details:
Galaxy S7 Edge

What is the nature of your issue
Bug

*Details

  1. Launch the wizard in the app
  2. Enter your zoneminder url with basic auth, username:password@domain. Make sure to enter a username and password with upper case and click next.
  3. On the next screen the app has put your username and password in lowercase so it fails to resolve on step 3 (look in the logs to see your password in lower case). You can work around it by retyping your username and password on step 2.

As a side note: I found this issue while trying to figure out why my camera was not showing any video on the app after successfully setting it up through the browser using basic auth. It seems my problem with no video is related to https://github.com/pliablepixels/zmNinja/wiki/FAQ#zmninja-is-suddenly-showing-red-xs-for-images---apis-are-working-zm-is-working-and-i-did-not-upgrade-zm-either. Not sure how often you update your FAQ, but this still seems like a problem on andriod with chrome 64.x. However, I did not see red X's, just three green dots across my video on the montage page and a black screen in live view (and it looks like the video keeps trying to refresh). Removing basic auth fixes it.

@pliablepixels
Copy link
Member

How are you entering your user:pass? You should be just entering the portal URL in the first step in the Wizard (do not enter user:pass@portal here)

In the next step, enable "I use authentication" and then enable "I use basic authentication" and enter user:pass

I tried that and it does not convert the user:pass to lower case. If you entered the user:pass as part of the portal, it would convert to lowercase.

Yes, I very recently replaced the "red X" with the green dots - will fix the FAQ - thanks for pointing it out

@jmzagorski
Copy link
Author

jmzagorski commented Feb 12, 2018

You are right, it does not convert to lowercase when you enter user:pass on step 2, only when you do that as part of step 1 as user:pass@portal. I did user:pass@portal on step 1 because the app auto-filled the username and password under "I use basic authentication" and I had my entire url on my clipboard (so this was easier for me). However, based on your response when user:pass@portal is entered on step 1, transforming to lowercase is expected and the way I did it is not a supported feature?

@pliablepixels
Copy link
Member

Fixed

@pliablepixels pliablepixels changed the title Wizard transforms basic auth credentials to lowercase Wizard transforms basic auth credentials to lowercase when entered as part of portal URL Feb 13, 2018
@jmzagorski
Copy link
Author

Thank you for such great support!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants