search

BanchanArt / banchan

The Co-Operative Commissions Platform
https://banchan.art
Other
169 stars 31 forks source link

fix how we pull in pfps when creating oauth users #791

Closed zkat closed 7 months ago

zkat commented 7 months ago

When the oauth service did not include a file extension in its url, which is a totally reasonable thing to do, we would just... fail to create the user altogether.

This PR does two things:

  1. Makes sure user creation happens regardless of pfp processing.
  2. Uses Image to load the raw binary data for the oauth pfp and always write it out as jpg.
coveralls commented 7 months ago

Coverage Status

coverage: 45.247% (+0.01%) from 45.234% when pulling 270c62316ec4f792c1f7c90f33b1b00d94b527b9 on zkat/fix-oauth-pfp-bug into 4efda46a7bc1141f9b9e8d1f830d90666c9f1d90 on main.