Skip to content

Update security.txt expires field - #34522

Merged
nickvergessen merged 1 commit into
masterfrom
bugfix/34480/update-security.txt-expires-field
Oct 11, 2022
Merged

Update security.txt expires field#34522
nickvergessen merged 1 commit into
masterfrom
bugfix/34480/update-security.txt-expires-field

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

@nickvergessennickvergessen added the 3. to review Waiting for reviews label Oct 11, 2022
@nickvergessennickvergessen added this to the Nextcloud 26 milestone Oct 11, 2022
@nickvergessen
nickvergessen requested review from a team, julien-nc and miaulalalaOctober 11, 2022 06:22
@nickvergessennickvergessen self-assigned this Oct 11, 2022
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/34480/update-security.txt-expires-field branch from d789d56 to 46070cfCompareOctober 11, 2022 06:26
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable25

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable24

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@nickvergessennickvergessen mentioned this pull request Oct 11, 2022
17 tasks

$response = "Contact: https://hackerone.com/nextcloud
Expires: 2021-12-31T23:00:00.000Z
Expires: 2023-04-31T23:00:00.000Z

@BartG95BartG95Nov 3, 2022

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

April has thirty days, not thirty-one....

This makes various security.txt checkers, such as internet.nl, tell me that the expires field is invalid.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Ouch, that's right. Unluckily the next maintenance releases just went out, so a fox will have to wait until in 4 weeks

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Fix in #34963

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Update security.txt Expires field

4 participants

@nickvergessen@tcitworld@BartG95@szaimen