apache/openwhisk

do not base64 encode __ow_body for raw web actions

Open

#3.920 aberto em 30 de jul. de 2018

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)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)).

Guia do colaborador

Pilha de tecnologia
None
Domain
backendcloud
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
fresh
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Understand OpenWhisk action encodingFamiliarity with raw web actions
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60
Direção de pesquisa
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`.