bhearsum@mozilla.com (:bhearsum) Reporter
	Description • 17 years ago

sdwilsh pointed out that the try server validation is a PITA if you have a huge patch. He also offered to write some client side validation into it.

Shawn, the code lives in http://lxr.mozilla.org/mozilla/source/webtools/buildbot-try. You'll need the $ALLOWED_* regexs from sendchange.cgi. Please move them to sendchange-ui.pm and import them back into sendchange.cgi.

Thank you very much for doing this!

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 1 • 17 years ago

Hey Shawn, are you still planning to do this?

	Shawn Wilsher :sdwilsh
	Comment 2 • 17 years ago

Yup - Firefox blockers are a bit higher on the priority list at the moment though ;)

Feel free to take this if you think it's important enough.

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 3 • 17 years ago

NP! Just checking :)

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 4 • 17 years ago

Moving try server bugs to the new component.

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 5 • 17 years ago

Hey Shawn, seems like you're too busy to do this right now. That's fine, I'm just going to reassign for now though.

	Shawn Wilsher :sdwilsh
	Comment 6 • 17 years ago

yeah, still got blockers :/

Once those are done I'll try to pick this back up.

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 7 • 16 years ago

Mass change of target milestone.

	bhearsum@mozilla.com (:bhearsum) Reporter
	Comment 8 • 16 years ago

Validation of the 'patch' field was done in 437143. Should/could add validation to other fields as well, so I'll leave this bug open.

	Chris Cooper [:coop] (he/him)
	Comment 9 • 15 years ago

Mass move of bugs from Release Engineering:Future -> Release Engineering. See
http://coop.deadsquid.com/2010/02/kiss-the-future-goodbye/ for more details.

	Lukas Blakk [:lsblakk] use ?needinfo Assignee
	Comment 10 • 14 years ago

The hg commit message patches went in today. Next we will be extending that to be able to read from a committed config file too. 

At this time there is no try server web UI form, and based on all the work done above, its unclear if we will ever need to create one. If we do, then we can do the validation at the same time as creating the UI. However, at this time, this bug's request is unnecessary.