Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Add checkbox to not post real passwords #16932

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

StefanSchoof
Copy link
Contributor

Hatte gerade die Idee, dass dies vielleicht den einen oder anderen abhalten könnte echte Passwörter oder sponsortoken zu posten.

@andig andig requested a review from naltatis October 28, 2024 15:06
@andig
Copy link
Member

andig commented Oct 28, 2024

People don't read. It would probably be better to hint on

evcc discuss

and/or integrate raising tickets directly from evcc.

@StefanSchoof
Copy link
Contributor Author

Ja, Menschen lesen nicht. Aber vielleicht gibt es paar den das Hilft und es ist ja deutlich kleiner als Tickets direkt aus evcc zu erzeugen.

War nur eine spontane Idee. Wenn die euch nicht gefällt macht das gerne zu.

@premultiply
Copy link
Member

Ich finde es gut.
Glaube zwar auch nicht dass es in allen Fällen hilft aber vielleicht wenigstens eine kleine Verbesserung bringt.

Es gibt ja auch noch weitere praktische aber leider kaum genutzte Hilfsmittel wie z.B. evcc dump --cfg was gleich die möglichen Konfig-Geheimnisse redigiert... 🤔

Vermutlich kommen wir da aber leider erst wirklich in der Sache weiter wenn mal irgendwann alles über die UI läuft.

@andig
Copy link
Member

andig commented Oct 31, 2024

Lass uns den Hinweis gerne in Fett oben drüber schreiben, aber bitte keine weitere Checkbox.

@StefanSchoof
Copy link
Contributor Author

Okay, ich kann in ein paar Tagen einen Vorschlag machen.

@andig andig added the infrastructure Basic functionality label Nov 1, 2024
@naltatis
Copy link
Member

naltatis commented Nov 2, 2024

Da es ja immer wieder die gleichen Strukturen sind könnten wir auch eine GH-Action einbauen, die Zeile für Zeile mit Regex über den Body eines neuen Issues läuft und ggf. sofort Feedback gibt. Müsste man vorher sicher in nem Test-Repo mal ausprobieren.

@andig andig marked this pull request as draft November 3, 2024 09:46
@StefanSchoof
Copy link
Contributor Author

Ein Regex ist schwierig, weil manchmal Nutzer für Passwörter einfach xxx oder [email protected] machen.

Für die Sponsortoken könnte das gehen. Erkennen und automatisch sperren. Aber dafür kenne ich eure Infrastructure zu wenig.

Sieht jetzt so aus
grafik
In Browser unter https://github.com/StefanSchoof/evcc/issues/new?assignees=&labels=&projects=&template=bug_report.yaml

@github-actions github-actions bot added the stale Outdated and ready to close label Nov 14, 2024
@premultiply premultiply removed the stale Outdated and ready to close label Nov 16, 2024
@github-actions github-actions bot added the stale Outdated and ready to close label Nov 23, 2024
@StefanSchoof
Copy link
Contributor Author

Was soll hiermit passieren? Gibt es Änderungsbedarf, kann das rein oder soll es geschlossen werden?

@github-actions github-actions bot removed the stale Outdated and ready to close label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Basic functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants