inducer / inducer/relate

Page POST data should include page ID

Open
#730 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
435
Forks
124
Avg merge
5h 40m
Merged PRs (30d)
14

Description

Currently, if a flow is updated between a page GET and the corresponding POST submission, submitted data may end up with a page for which it is not intended (e.g. on a page that doesn't allow form submission, if the original page did allow it). The core issue here is that pages are identified by their page index in the flow, but the mapping of the index to pages may change over time and is thus not robust. Including the page ID with the page submit POST data would allow catching and rejecting these mismatched POSTs and rejecting them with a useful error message (asking the user to resubmit in the correct place in the updated flow).

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.