Skip to content
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 ADR035: offer Amazon S3 for form submissions #179

Merged
merged 5 commits into from
Oct 16, 2024

Conversation

james-sheppard-gov-uk
Copy link
Contributor

@james-sheppard-gov-uk james-sheppard-gov-uk commented Oct 8, 2024

Rendered view

PR Checklist

  • If you are proposing a new decision record document, used the right template for that
    - (ADR, decision-record, engagement, research)
  • Set yourself as the Assignee
  • Tag anyone you would like to review, or @forms-design or @forms-devs
  • Fill in the template below

What

A new ADR to allow form submissions to be sent to Amazon S3.

How to review

  1. Semantic: Do you agree with the changes?
  2. Syntactic: Spelling, grammar, etc.

Who can review

Anyone, but in particular those who worked on the Amazon S3 spike.

@james-sheppard-gov-uk james-sheppard-gov-uk self-assigned this Oct 8, 2024
@james-sheppard-gov-uk james-sheppard-gov-uk changed the title Add ADR035: offer Amazon s3 for form submissions Add ADR035: offer Amazon S3 for form submissions Oct 8, 2024
chao-xian
chao-xian previously approved these changes Oct 8, 2024
Copy link
Contributor

@chao-xian chao-xian left a comment

Choose a reason for hiding this comment

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

Ah thank you James!

@lfdebrux
Copy link
Member

lfdebrux commented Oct 9, 2024

I worry about what happens if the team managing the S3 bucket accidentally changes the bucket access or configuration in a way that stops us being able to upload documents to it... what will happen to form submissions if the S3 upload doesn't work?

@DavidBiddle
Copy link
Contributor

@lfdebrux there's a section on this in the spike doc:

To start with we can replicate the current behaviour with email submissions via Notify, where an error message is shown to the form submitter and they are obliged to resubmit.

User research has also suggested that some users would like to receive an email notification that submission failed. This would be basic to start with and the call to action would just be to contact the GOV.UK Forms team. This implies we might want to allow configuration of a support email, to which submission failure notifications would be sent.

In future we could look at temporarily storing submissions so that they can be processed asynchronously. This would provide robustness against submission failures for all submission types (e.g. Notify and SES).

This would allow retries for failed submissions, and would retain failed submission data for a time to allow inspection and replaying submission. This would also allow form fillers to continue submitting forms, even if the submission pipeline isn’t working.

@james-sheppard-gov-uk do you think it's worth incorporating any of that information in here?

@james-sheppard-gov-uk
Copy link
Contributor Author

Thanks for your comments @lfdebrux and @DavidBiddle - I've made a few additions, please could you take another look?

Copy link
Contributor

@DavidBiddle DavidBiddle left a comment

Choose a reason for hiding this comment

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

Looks good to me 🎉

@james-sheppard-gov-uk james-sheppard-gov-uk merged commit 9129ca9 into main Oct 16, 2024
@james-sheppard-gov-uk james-sheppard-gov-uk deleted the adr-use-amazon-s3-for-submissions branch October 16, 2024 10:12
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.

4 participants