apache/openwhisk

do not base64 encode __ow_body for raw web actions

Open

#3 920 ouverte le 30 juil. 2018

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Scala (6 777 stars) (1 177 forks)batch import
controllergood first issuehelp wanted

Description

#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)).

Guide contributeur

Stack technique
Aucun
Domaine
backendcloud
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Understand OpenWhisk action encodingFamiliarity with raw web actions
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
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`.