hardcoded paths, and hardcoded filenames
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Anfängerfreundlichkeit
- 20/100
Rechercherichtung
Beginnen Sie mit der Untersuchung von cron_wrapper, virtual_servers, dem Pfad pythonvenv/python37 sowie den Konfigurationsdateien .f5creds und load-balancer. Verfolgen Sie, wie Pfade, Dateinamen, Zugangsdaten und F5_HTTPS ausgewählt werden, und prüfen Sie anschließend, ob Überschreibungen über die Befehlszeile oder mehrere load balancers unterstützt werden. Als erledigt gilt die Aufgabe, wenn ein definierter Konfigurationsansatz vorhanden ist, der die gemeldeten hartcodierten Annahmen vermeidet und das erforderliche HTTP challenge- und HTTPS deployment-Verhalten beibehält.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
cron_wrapper is not executable in the repo. That should be changed.
paths are hardcoded which won't work for most users. Perhaps something like:
- cd $HOME/lets-encrypt-python
Also, I don't have a
pythonvenv/python37/bin/activate
what should that contain?
This code breaks our setup. I have multipe file sets.
- {loadbalancer} - contains all my setting including host, user, pass, http, https
- {loadbalancer}.cfg - DOMAINS_TXT="${BASEDIR}/{load balancer}.dom"
- {loadbalancer}.dom - all the san certs each set on one line
Having the authentication in the environment meant that I could talk to different load-balancers, potentially with different credentials all in one directory. the ONE .f5creds that can't seem to be overridden breaks that.
The file "virtual_servers" also breaks this setup, looks hardcoded, and can't be overridden on the command line. This file assumes that the virtual server name for a given hostname is the same on each load balancer, which is often not a valid assumption. It also require adding each domain on a san cert to the lookup table. This might be useful in some situations, but not for me at the moment as I'm only testing on one vs per lb.
Eventually it would be nice to be able to deploy the validation changes on multiple load balancers, test, and then remove from multiple load balancers in the case were a host uses GTM/DNS load balancing or just has multiple IPs in DNS.
This seems to remove the F5_HTTPS handling. The Let's Encrypt challenge is over http, which should be a separate vip from the one where the cert should be installed in the end. We'll want a F5_HTTPS if eventually we want to add the created ssl profile to a vip, as it won't be the same vip.
- Vorherrschende Sprache
- Shell
- Sterne
- 50
- Forks
- 12
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus f5devcentral/lets-encrypt-python
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
-
Cert Chain Exists Offen
f5devcentral/lets-encrypt-python#2 · 2 Kommentare · 1 zugewiesene Person ·
Alle Issues in f5devcentral/lets-encrypt-python
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
community-scripts/ProxmoxVE#17425 · 1 Kommentar ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 90/100
danielmiessler/LifeOS#2218 ·
-
docs(agents): strengthen the no-backslash-escaped-backticks rule with an issue-creation example Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 92/100
-
technical-debt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
ll7/robot_sf_ll7#9560 ·
-
Update ghgrab to 2.1.0 Offenpackage-update
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
oSoWoSo/vOid_Community_repOsitory#148 · 1 Kommentar ·