Skip to content

Conversation

@jpdillingham
Copy link
Member

@jpdillingham jpdillingham commented Apr 7, 2025

Expands on #1303

Sorry @dkschruteBeets for hacking this up so much; I try to do validation at startup and then fail gracefully later on (also minimize lines of code where possible)

Testing

  • Works if no file is specified
  • Fails to start if non-existing file is specified
  • Logs warning and returns picture-less response if file is moved after startup
  • Works with .jpg
  • Works with .png (Soulseek NS excluded)
  • Works with .gif
  • Works with .bmp

I think whether it works with a particular format is a function of the client on the other end, so I'm not sure

@jpdillingham jpdillingham merged commit f0ab25a into master Apr 7, 2025
4 checks passed
@jpdillingham jpdillingham deleted the profile-pic branch April 7, 2025 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants