Skip to content

fix fileupload hmac validation, handle multi or single property value…#33

Open
yodaXX wants to merge 1 commit into
WapplerSystems:release/v13from
yodaXX:fileupload
Open

fix fileupload hmac validation, handle multi or single property value…#33
yodaXX wants to merge 1 commit into
WapplerSystems:release/v13from
yodaXX:fileupload

Conversation

@yodaXX

@yodaXX yodaXX commented Feb 27, 2026

Copy link
Copy Markdown
  1. Adds ResourcePointer prefix for hmac validation in UploadedFileReferenceConverter
  2. Adjusted UploadedResourceViewHelper to resolve uploaded resources if multiupload is not enabled

fixes #32

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm, that this change fixes #32

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know, what this change fixes? Is it also related to #32?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I remember correctly I discovered some problems with the Upload field not resolving/showing the previous uploaded file when the same form step is shown again. It was with the multiple option disabled.
These changes should handle single or multiupload the same way
I was testing a multistep form at that time.

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

Successfully merging this pull request may close these issues.

UploadedFileReferenceConverter: Use of hashService requires an additional secret

2 participants