appsembler / edx-platform

Appsembler Tahoe fork of Open edX. Branches: `main` for Juniper and `hawthorn/main` for Hawthorn.
https://appsembler.com/products/tahoe/
GNU Affero General Public License v3.0
13 stars 15 forks source link

fix AttributeError handling in TahoeUserMetadataProcessor #1327

Closed bryanlandia closed 1 year ago

bryanlandia commented 1 year ago

Change description

AttributeErrors can be raised with some weird events that might have a String as event_data instead of a dict. Had sloppy code inside the try/except before. Inner exception was still handled caught by event tracking pipeline but good to fix this to keep logs quieter

Type of change

Related issues

Checklists

Development

Security

Code review

github-actions[bot] commented 1 year ago

Checking git merge conflicts against https://github.com/edx/edx-platform.git

Comparing with open-release/nutmeg.master
Benchmark conflicts with main 292
Current conflicts 292
Summary Good work! No added conflicts.
Comparing with master
Benchmark conflicts with main 300
Current conflicts 300
Summary Good work! No added conflicts.