aws / aws/base64io-python

Feature Request: Add support for altchars

Offen
#40 0 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
42
Forks
29
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

First, thanks for a cool package!

The core function that this package turns into a streaming interface is `base64.b64encode(s, altchars=None)`, and that function supports alternate characters for `+` and `/`:

> Optional altchars must be a bytes-like object of at least length 2 (additional characters are ignored) which specifies an alternative alphabet for the + and / characters. This allows an application to e.g. generate URL or filesystem safe Base64 strings. The default is None, for which the standard Base64 alphabet is used.

Seems like it would be fairly simple to support alt chars in this package by adding a new, optional parameter to the `Base64IO` constructor.

Happy to make a PR to add the feature if it's something the maintainers would want.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, den Konstruktor von Base64IO und die Streaming-Pfade zu finden, die Python's base64.b64encode(s, altchars=None) umschließen. Verfolge sowohl das Kodierungs- als auch das Dekodierungsverhalten, bevor du entscheidest, wie der optionale Parameter weitergereicht werden soll. Als abgeschlossen gilt die Aufgabe, wenn Base64IO alternative Zeichen für + und / akzeptiert und dabei das Standardverhalten beibehält; füge die relevanten Tests hinzu oder aktualisiere sie, falls solche vorhanden sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
backend
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.