Bug 245674

Summary: security/py-certbot: allow more options for restarting services after certificate renewals
Product: Ports & Packages Reporter: Dmitry Marakasov <amdmi3>
Component: Individual Port(s)Assignee: Danilo G. Baio <dbaio>
Status: Closed FIXED    
Severity: Affects Many People CC: dbaio, fjoe, python
Priority: --- Flags: dbaio: maintainer-feedback+
dbaio: merge-quarterly+
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://reviews.freebsd.org/D25391
See Also: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=245954
Attachments:
Description Flags
Patch none

Description Dmitry Marakasov freebsd_committer freebsd_triage 2020-04-16 20:01:24 UTC
Created attachment 213468 [details]
Patch

In addition to stop service → renew cert → start service pattern, support restarting and reloading a service after certificate renewals, which is more common and useful pattern which implies less/no downtime.
Comment 1 Max Khon freebsd_committer freebsd_triage 2020-05-25 04:25:36 UTC
There should also be support for deploy hook (see also #245954)
Comment 2 Danilo G. Baio freebsd_committer freebsd_triage 2020-06-21 18:04:15 UTC
Please see review D25391, I tried to gather all scenarios in the periodic script.
Comment 3 commit-hook freebsd_committer freebsd_triage 2020-07-11 12:34:19 UTC
A commit references this bug:

Author: dbaio
Date: Sat Jul 11 12:33:24 UTC 2020
New revision: 541966
URL: https://svnweb.freebsd.org/changeset/ports/541966

Log:
  security/py-certbot: Improve periodic script

  This change will keep the default behavior in the periodic script
  and will add options to customize each parameter for those who want to:
   - weekly_certbot_pre_hook
   - weekly_certbot_post_hook
   - weekly_certbot_deploy_hook
   - weekly_certbot_custom_args

  PR:		245674, 245954
  Reported by:	amdmi3, fjoe
  Reviewed by:	koobs
  Approved by:	dbaio, koobs (python, maintainer)
  MFH:		2020Q3
  Differential Revision:	https://reviews.freebsd.org/D25391

Changes:
  head/security/py-certbot/Makefile
  head/security/py-certbot/files/500.certbot.in
  head/security/py-certbot/files/pkg-message.in
  head/security/py-certbot/pkg-message
Comment 4 Danilo G. Baio freebsd_committer freebsd_triage 2020-07-11 12:37:34 UTC
Committed, waiting MFH.
Comment 5 commit-hook freebsd_committer freebsd_triage 2020-07-11 12:50:24 UTC
A commit references this bug:

Author: dbaio
Date: Sat Jul 11 12:50:21 UTC 2020
New revision: 541967
URL: https://svnweb.freebsd.org/changeset/ports/541967

Log:
  MFH: r541966

  security/py-certbot: Improve periodic script

  This change will keep the default behavior in the periodic script
  and will add options to customize each parameter for those who want to:
   - weekly_certbot_pre_hook
   - weekly_certbot_post_hook
   - weekly_certbot_deploy_hook
   - weekly_certbot_custom_args

  PR:		245674, 245954
  Reported by:	amdmi3, fjoe
  Reviewed by:	koobs
  Approved by:	dbaio, koobs (python, maintainer)
  Differential Revision:	https://reviews.freebsd.org/D25391

  Approved by:	ports-secteam (joneum)

Changes:
_U  branches/2020Q3/
  branches/2020Q3/security/py-certbot/Makefile
  branches/2020Q3/security/py-certbot/files/500.certbot.in
  branches/2020Q3/security/py-certbot/files/pkg-message.in
  branches/2020Q3/security/py-certbot/pkg-message