apache/openwhisk

do not base64 encode __ow_body for raw web actions

Open

#3.920 geöffnet am 30. Juli 2018

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (6.777 Stars) (1.177 Forks)batch import
controllergood first issuehelp wanted

Beschreibung

#3919 removes the base64 encoding requirement when returning JSON as string (and setting the content-type). However we continue to encode JSON for raw http actions receiving __ow_body. For symmetry, we should stop doing that as well. This is largely in place for legacy reasons (spray vs akka). The net of this change is that functions would JSON.parse(param.__ow_body) instead of JSON.parse(decode64(param.__ow_body)).

Contributor Guide

Tech Stack
Keine
Domain
backendcloud
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Understand OpenWhisk action encodingFamiliarity with raw web actions
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
The issue references PR #3919 that removed base64 encoding for JSON responses. The goal is to similarly remove base64 encoding for raw web actions' ow body. Look at the code handling raw HTTP actions, likely in the same area as the previous change. The comment mentions spray vs akka legacy reasons, so the change might be in the routing or serialization layer. Search for 'base64' or 'decode64' in the codebase related to ` ow body`.