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

chore: remove eb shift frontend feature flags #6869

Merged
merged 10 commits into from
Nov 14, 2023

Conversation

tshuli
Copy link
Contributor

@tshuli tshuli commented Nov 6, 2023

Problem

Works towards FRM-1232 and FRM-1413

  • This PR removes the frontend feature flags for (i) encryption boundary shift and (ii) virus scanner. Henceforth, all frontend traffic will be routed through the new storage mode endpoint and virus scanner.
  • This is in preparation for myinfo storage mode feature.

Tests

  • Create storage mode form. Open in public form view.
  • Submission with attachment works
  • Attachment is uploaded to s3
  • Malicious attachment is blocked
    secure.eicar.org_eicar.com.txt

Copy link
Contributor

@wanlingt wanlingt left a comment

Choose a reason for hiding this comment

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

generally lgtm!

I also searched FRM-1232 and FRM-1413 in the codebase to check for other TODOs in the frontend. In createSubmission.ts, we can remove filterSendableStorageModeResponses and the function that it's used in (createEncryptedSubmissionData). I realised from there that we're still using the old /encrypt endpoint for preview mode - it's non-blocking for this PR (only when we remove the backend endpoints) but just to note for next time.

also a nit: should we remove "Clear" from the function names - e.g. submitStorageModeClearFormWithVirusScanningWithFetch? I don't think it adds much since now there's no distinction between the modes to submit a storage mode form

Copy link

linear bot commented Nov 8, 2023

FRM-1413 Remove virus scanning feature flag

Description

Remove virus scanning feature flag once virus scanner integration has been completely rolled out.

Search for TODO (FRM-1413) in codebase

FRM-1232 Remove old storage submission endpoint & encryption boundary shift feature flag

Description

Remove old storage submission endpoint to cleanup codebase.

@tshuli
Copy link
Contributor Author

tshuli commented Nov 8, 2023

thanks @wanlingt ! will go thru the comments :)

There is an existing ticket to implement the encryption boundary shift for preview mode, but as you said, non-blocking for now:
https://linear.app/ogp/issue/FRM-1506/encryption-boundary-shift-for-storage-mode-form-submissions-previews

@wanlingt
Copy link
Contributor

wanlingt commented Nov 8, 2023

thanks @wanlingt ! will go thru the comments :)

There is an existing ticket to implement the encryption boundary shift for preview mode, but as you said, non-blocking for now: https://linear.app/ogp/issue/FRM-1506/encryption-boundary-shift-for-storage-mode-form-submissions-previews

oh nice thanks @tshuli ! Just added it to the Shift Encryption Boundary project on linear

@tshuli tshuli enabled auto-merge (squash) November 14, 2023 15:26
@tshuli tshuli merged commit d5bbb4d into develop Nov 14, 2023
20 of 22 checks passed
@tshuli tshuli deleted the chore/remove-eb-shift-feature-flag branch November 14, 2023 15:46
tshuli added a commit that referenced this pull request Nov 17, 2023
tshuli added a commit that referenced this pull request Nov 17, 2023
chore: Revert remove eb shift frontend feature flags (#6869)
@tshuli tshuli mentioned this pull request Nov 20, 2023
12 tasks
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.

None yet

2 participants