Adding padding to base64 encoded policy decisions (#1738)

Switching to standard encoding Signed-off-by: Matthew A Johnson <matjoh@microsoft.com>
microsoft · Apr 20, 2023 · d816cbe · d816cbe
1 parent 1143934
commit d816cbe
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pkg/securitypolicy/securitypolicy.go b/pkg/securitypolicy/securitypolicy.go
@@ -146,7 +146,7 @@ func ExtractPolicyDecision(errorMessage string) (string, error) {
 		return "", errors.Errorf("unable to extract policy decision from error message: %s", errorMessage)
 	}
 
-	errorBytes, err := base64.RawURLEncoding.DecodeString(matches[1])
+	errorBytes, err := base64.StdEncoding.DecodeString(matches[1])
 	if err != nil {
 		return "", err
 	}

diff --git a/pkg/securitypolicy/securitypolicyenforcer_rego.go b/pkg/securitypolicy/securitypolicyenforcer_rego.go
@@ -352,7 +352,7 @@ func (policy *regoEnforcer) policyDecisionToError(ctx context.Context, decision
 
 	log.G(ctx).WithField("policyDecision", string(decisionJSON))
 
-	base64EncodedDecisionJSON := base64.RawURLEncoding.EncodeToString(decisionJSON)
+	base64EncodedDecisionJSON := base64.StdEncoding.EncodeToString(decisionJSON)
 	errorMessage := fmt.Errorf(policyDecisionPattern, base64EncodedDecisionJSON)
 	if policy.maxErrorMessageLength == 0 {
 		// indicates no message truncation
@@ -373,7 +373,7 @@ func (policy *regoEnforcer) policyDecisionToError(ctx context.Context, decision
 			log.G(ctx).WithError(err).Error("unable to marshal error object")
 			decisionJSON = []byte(`"Unable to marshal error object"`)
 		}
-		base64EncodedDecisionJSON = base64.RawURLEncoding.EncodeToString(decisionJSON)
+		base64EncodedDecisionJSON = base64.StdEncoding.EncodeToString(decisionJSON)
 		errorMessage = fmt.Errorf(policyDecisionPattern, base64EncodedDecisionJSON)
 
 		if len(errorMessage.Error()) <= policy.maxErrorMessageLength {