zoneminder icon indicating copy to clipboard operation
zoneminder copied to clipboard

When editing a monitor, using RTSP, host cannot contain random punctuation

Open connortechnology opened this issue 10 years ago • 9 comments

However the password might have punctuation in it.

We need to be consistent on all the various methods, including the control tab.

I think username/password should be separate fields, that way we can actually do useful input validation.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

connortechnology avatar Feb 05 '15 20:02 connortechnology

Use Username and Password fields already in the db and just do this.

SteveGilvarry avatar Mar 18 '17 10:03 SteveGilvarry

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jun 15 '18 21:06 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Sep 28 '18 21:09 stale[bot]

I hate you stale bot.

connortechnology avatar Sep 30 '18 14:09 connortechnology

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Nov 29 '18 14:11 stale[bot]

Just commenting because I've reached this point, but need sleep (sorry). Hopefully this comment will keep the stale-bot at bay for a little longer.

GlenCooper avatar Nov 04 '19 02:11 GlenCooper

Hi all, i'm working on this and i've just got the system working from source. I'll let you know as i have a change ready.

parvit avatar Jul 17 '22 06:07 parvit

@connortechnology I've added the PR responding to your request.

parvit avatar Jul 22 '22 11:07 parvit

Hello, is there any feedback on this?

parvit avatar Jul 28 '22 08:07 parvit

@connortechnology Thanks for merging the PR, if we may close this ticket it would allow me to claim the bounty.

As a side note: I would like to work also on #355, but it's locked so i don't know if i can proceed, can we unlock that one? thanks.

parvit avatar Aug 22 '22 06:08 parvit

I don't see anything locked about #355. You might want to discuss your approach on it before digging in too far. Would be great to have it done though.

I've modified your code a bit. in php8 passing null to a lot of functions generates errors. Also I'm not sure strlen is the most efficient way to determine if a string is empty or not. I also took out the setting of auth_type in ffmpeg. It shouldn't be necessary and is just generating errors in the logs.

connortechnology avatar Aug 22 '22 18:08 connortechnology

This is what i see from my side on #355 image

parvit avatar Aug 22 '22 19:08 parvit

Okay it is unlocked.

connortechnology avatar Aug 22 '22 19:08 connortechnology

It worked, thanks.

parvit avatar Aug 22 '22 19:08 parvit