1
100%
main: 100%

Ran 06 Jul 2023 11:22PM UTC

Files 38

Run time 0s

Badge

Embed ▾

Committed 06 Jul 2023 11:12PM UTC coverage: 100.0%. Remained the same

Job # 5481053858.1

Build Type

push

github

Committed by

mbland

Commit Message

TestHandleApiRequest: use POST unsubscribe

This is in preparation for allowing only POST unsubscribe requests.

The handler logic doesn't care what HTTP method is used to unsubscribe,
which is why this test change didn't require any code changes. The
/unsubscribe HTTP method is enforced by the CloudFormation template.

Still, it's good for the test input to closely resemble what's expected
from the configured API Gateway, even if it doesn't affect the handler
logic.

I mean, I could add logic to enforce the expected HTTP method for each
API endpoint. Maybe that's not a bad idea. However, it will have to come
after updating the system to point {{UnsubscribeUrl}} at a page with a
web form.

That, in fact, is why I'm diving into making sure `@` is encoded as
`%40` and thinking of eliminating GET /unsubscribe. I accidentally
unsubscribed myself by clicking on the unsubscribe link in an email.
Updating {{UnsubscribeUrl}} to point to a form will allow the user to
make sure they really intend to unsubscribe. This POST /unsubscribe will
be less error prone while also being more semantically correct. (I think
technically /verify should also be a POST, but it seems conventional to
have such links always use GET.)

Run Details

2240 of 2240 relevant lines covered (100.0%)

15.97 hits per line

mbland / elistman / 5481053858 / 1
100%
main: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5481053858.1

mbland / elistman / 5481053858 / 1 100% main: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 5481053858.1

mbland / elistman / 5481053858 / 1
100%
main: 100%

README BADGES
x