Skip to content

Gpsd + Web Auth Cedar Crash fix#386

Merged
brickbots merged 2 commits intomainfrom
gpsd_fix
Feb 28, 2026
Merged

Gpsd + Web Auth Cedar Crash fix#386
brickbots merged 2 commits intomainfrom
gpsd_fix

Conversation

@brickbots
Copy link
Owner

This messy compound PR fixes two issues:

  1. Original dongles used in v1 PiFinders never report satellite info. The asynchronous handling in the gps_gpsd code would block waiting for these reports as the GPSD module itself is not asynchronous.
  • Switched code back to synchronous and merged satellite and position message handling to avoid blocking for older GPS dongles
  1. The cedar detect GRPC calls have a high likelihood of crashing when the web interface shells out to check user credentials. After debugging and investigation with an end user, authentication on the web interface seems to be one of the primary causes of solver crashing.
  • Switched auth checking to use the pam module which is a much better solution overall. Hopefully this will prevent whatever is causing the GRPC calls to fail during/after the auth process

@brickbots brickbots mentioned this pull request Feb 28, 2026
@brickbots brickbots merged commit 0c14187 into main Feb 28, 2026
2 checks passed
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.

1 participant