-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add validation for a few inputs (like flow and uuid) in a few spots. #548
Conversation
* | ||
* @param flow {@link String} of a flow name. | ||
* @return Returns a {@link FlowConfiguration} object. | ||
* @throws ResponseStatusException when FlowConfigurations are not found. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are auto formatting corrections. I'm not this picky. :)
834be7c
to
802ebdd
Compare
<div> | ||
<th:block | ||
th:replace="~{fragments/inputs/text :: | ||
<div th:replace="~{fragments/toolbar :: toolbar}"></div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are no actual changes in this file, other then format ones. I can reverse this, but since we are moving to these new spacings anyways, I'll just leave this.
Map<String, Object> iteration = submission.getSubflowEntryByUuid(screen.getSubflow(), uuidToVerify); | ||
if (iteration == null) { | ||
throwNotFoundError(submission.getFlow(), screen.getName(), | ||
String.format("UUID not found in iterations for subflow '%s' in flow '%s', when navigating to '%s'", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we log the UUID?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would be useful. I'll add it in.
802ebdd
to
ca87c29
Compare
Issue tracking number π
Description of change βοΈ
This adds validation of some request inputs in a few more spots. This is to ensure that "flow" and "uuid" are not used w/o validating they are legitimate data.
Priority π₯
Effect on other applications using FFB π
This should not affect any system, it will just make us a little more secure.
Testing
β Checklist before requesting a review
style?