-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for Protobufs in CloudEvents #391
Comments
We have no wired up proto support. Is this necessary? |
I think it is @kevmoo. The new events from Firestore to trigger changes directly without cloud audit are only available with proto. |
Let me dig in there, @diezep ! |
@diezep – can you point me to documentation from cloud run about when/where/how to expect |
To be honest, I already searched for that before and didn't find anything about it. Not even how to handle that type of content. |
Closes #391 Also improve efficiency of JSON decoding
Closes #391 Also improve efficiency of JSON decoding
@diezep – see https://github.com/GoogleCloudPlatform/functions-framework-dart/tree/main/examples/protobuf_firestore ! 😄 (The readme was copy-pasted and not updated. Need to get on that!) |
I get this error when i trigger a cloud event in Cloud Run using application/protobuf as a event data content type
Bad request. Unsupported encoding "application/protobuf". Only "application/json" is supported. package:functions_framework/src/json_request_utils.dart 42 mustBeJson package:functions_framework/src/targets/cloud_event_targets.dart 90 _decodeBinary package:functions_framework/src/targets/cloud_event_targets.dart 66 _eventFromRequest package:functions_framework/src/targets/cloud_event_targets.dart 53 CloudEventWithContextFunctionTarget.handler package:functions_framework/src/run.dart 49 _forbiddenAssetMiddleware.<fn> package:functions_framework/src/logging.dart 171 cloudLoggingMiddleware.hostedLoggingMiddleware.<fn>.<fn> dart:async _CustomZone.runGuarded package:functions_framework/src/logging.dart 168 cloudLoggingMiddleware.hostedLoggingMiddleware
The text was updated successfully, but these errors were encountered: