Shopify / Shopify/mruby-engine

mgem questions

Offen
#23 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
C
Sterne
81
Forks
19
Ø Merge
12 T. 4 Std.
Gemergte PRs (30 T.)
1

Beschreibung

First of all, amazing project!

Now some questions:

  1. Any chance to move the mruby-mpdecimal into an own repository so that we can link it from our mgem-list (http://github.com/mruby/mgem-list)?

  2. I notice that you removed several methods from the mruby-time gem (e.g. local, now). I guess it is relevant from your security point of view, I would like to understand your thought about this change.

Furthermore thanks for your great contribution from this bug bounty. I guess it is somehow financed by Spotify. Nevertheless I think the security considerations are a really valuable addition. So thanks again!

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Überprüfe das Projekt mruby-mpdecimal und die in der Issue referenzierten Änderungen an mruby-time und prüfe anschließend den Kontext des Repositorys mruby/mgem-list. Eine vollständige Antwort würde die entfernten Methoden und die sicherheitsbezogene Begründung erläutern und feststellen, ob mruby-mpdecimal in ein eigenes Repository verschoben werden sollte.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
ruby
Bereich
security
Issue-Typ
Dokumentation
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
15/100

Neue Issues direkt in Ihr Postfach

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