ampproject / ampproject/amppackager

Add integration tests for ACME and OCSP renewals

Open
#433 0 comments 0 reactions 0 assignees View on GitHub
fixit
Dominant language
Go
Stars
141
Forks
46
PR merge metrics
No merged PRs in 30d

Description

There are a lot of corner cases with these things, since they are both concurrent, and on different schedules, and involve the filesystem. We should go through, try and understand what all the corner cases are, and write tests for them. Split off from #93.

Contributor guide

Open the contributing guide

Research direction

The issue names no files or tests; start by reviewing the renewal implementation and the context in #93. Identify corner cases involving concurrent ACME and OCSP schedules and filesystem state, then add integration coverage demonstrating those cases and confirming both renewal flows behave correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
security, testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.