Skip to content
This repository has been archived by the owner on Apr 16, 2022. It is now read-only.

On export, skip encrypted submissions with manifest but no .enc file. Fix #897

Open
wants to merge 1 commit into
base: v1.11.x
Choose a base branch
from

Conversation

anaskhl
Copy link

@anaskhl anaskhl commented Jan 19, 2021

…and mark as failed #895

Closes #

What has been done to verify that this works as intended?

Why is this the best possible solution? Were any other approaches considered?

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

Does this change require updates to documentation? If so, please file an issue at https://github.com/getodk/docs/issues/new and include the link below.

@anaskhl anaskhl changed the title On export, skip encrypted submissions with manifest but no .enc file … On export, skip encrypted submissions with manifest but no .enc file. Fix #897 Jan 19, 2021
@anaskhl anaskhl changed the title On export, skip encrypted submissions with manifest but no .enc file. Fix #897 On export, skip encrypted submissions with manifest but no .enc file. Fix Jan 19, 2021
@anaskhl
Copy link
Author

anaskhl commented Jan 19, 2021

When reading the list of submissions for parsing, A check is done if the form is encrypted, and submission.xml.enc is not available then log that the file is not available and exclude from the submissions.

@anaskhl
Copy link
Author

anaskhl commented Jan 19, 2021

It looks like the build failed because of java version issue. How can we fix this?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant