Skip to content

Commit 475aff2

Browse files
author
Curtis Mason
committed
lint fix and deepcopy for headers/data
Signed-off-by: Curtis Mason <cumason@google.com>
1 parent 13d295b commit 475aff2

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

cloudevents/sdk/event/base.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
'ce-time'
3232
}
3333

34+
3435
# TODO(slinkydeveloper) is this really needed?
3536
class EventGetterSetter(object):
3637

@@ -143,9 +144,11 @@ def UnmarshalBinary(
143144
body: typing.IO,
144145
data_unmarshaller: typing.Callable
145146
):
146-
# Extract only CloudEvent fields from headers
147-
ce_fields = [field for field in headers if \
148-
(field in _ce_required_fields) or (field in _ce_optional_fields)]
147+
# Extract only CloudEvent fields from headers
148+
ce_fields = [
149+
field for field in headers if
150+
(field in _ce_required_fields) or (field in _ce_optional_fields)
151+
]
149152

150153
headers = {key: headers[key] for key in ce_fields}
151154
for header, value in headers.items():

cloudevents/sdk/http_events.py

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -44,14 +44,11 @@
4444
import json
4545
import typing
4646

47-
from cloudevents.sdk import converters
4847
from cloudevents.sdk import marshaller
4948

5049
from cloudevents.sdk.event import base
5150
from cloudevents.sdk.event import v1
5251

53-
import requests
54-
5552

5653
class Event(base.BaseEvent):
5754
"""
@@ -114,8 +111,8 @@ def __init__(
114111
else:
115112
raise NotImplementedError
116113

117-
self.headers = headers
118-
self.data = data
114+
self.headers = copy.deepcopy(headers)
115+
self.data = copy.deepcopy(data)
119116
self.marshall = marshaller.NewDefaultHTTPMarshaller()
120117
self.event_handler = v1.Event()
121118
self.marshall.FromRequest(

0 commit comments

Comments
 (0)