Skip to content

Fix WeWork API request classification - #20

Merged
jeromewir merged 1 commit into
mainfrom
fix/wework-rate-limit
Sep 6, 2026
Merged

Fix WeWork API request classification#20
jeromewir merged 1 commit into
mainfrom
fix/wework-rate-limit

Conversation

@jeromewir

Copy link
Copy Markdown
Owner

Summary

  • send the browser request context expected by WeWork for property, location, and booking API calls
  • classify upstream 429 responses as rate limits instead of internal server errors
  • return HTTP 429 for rate-limited single and batch booking requests

Investigation

  • a location cache miss makes two WeWork API calls: property lookup, then location details lookup
  • the batch size does not multiply location lookups
  • the location endpoint returned 429 with Resty's default server headers and 200 with authenticated browser headers
  • the current Workspace listing endpoint returns 403 for this On Demand account and is not a compatible replacement

Verification

  • go test ./...
  • go vet ./...
  • git diff --check

@jeromewir
jeromewir merged commit 08a457b into main Sep 6, 2026
1 check 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