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

"Authentication Success" reported, even when not possible #847

Closed
dstulken opened this issue Aug 22, 2019 · 1 comment
Closed

"Authentication Success" reported, even when not possible #847

dstulken opened this issue Aug 22, 2019 · 1 comment

Comments

@dstulken
Copy link

Platform & OS Version

iOS 12.3.1

The version of zmNinja you are reporting:

1.3.073

The version and OS of ZoneMinder you are using:

Doesn't matter.  (see below).

Device details:

iPhone SE

What is the nature of your issue

Bug

Details

If zmNinja is configured with both ZM authentication and basic authentication disabled, then the app will display a "ZoneMinder authentication success" banner upon startup, regardless of whether any connectivity to the ZoneMinder server was actually established.

For example, in the attached screenshot, the iPhone device was in airplane mode with wifi disabled, yet zmNinja still reports a successful authentication. Clearly no communication, let alone authentication, could have been possible in this case, so the display of the status message is quite misleading - especially when followed moments later by the API Access Error messages. For situations where network connectivity is unreliable, for example, this can lead to a lot of head-scratching in determining where the failure is located, as it gives the impression that at least some level of communication with the ZoneMinder server had been established.

One possible solution to this bug would be to simply not display any authentication banners of any kind if both styles of auth are disabled in the app settings, instead reserving both success and failure prompts for when the user had at least one form of authentication enabled.

IMG_6887

zmNinja Debug Logs

Zoneminder Logs (if applicable)

(Not applicable)

Web Server Error Logs (if applicable, especially for streaming errors or API error)

(Not applicable)

Thanks!

@welcome
Copy link

welcome bot commented Aug 22, 2019

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you follow the issue template or I may not respond.

Simpler1 pushed a commit to Simpler1/zmNinja that referenced this issue Oct 25, 2022
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

No branches or pull requests

1 participant