Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ATProto/api — actor.GetProfiles suddenly defaulting to gzip output #2831

Open
scarnecchia opened this issue Sep 25, 2024 · 1 comment
Open
Labels
bug Something isn't working

Comments

@scarnecchia
Copy link

scarnecchia commented Sep 25, 2024

Until this morning, calls to actor.getprofiles that did not specify accept-encoding were returning plaintext. This morning, they began returning gzipped data.

Steps to reproduce the behavior:

  1. In bash, define a list of 25 dids as params (actor=did…&actor=did…)
  2. Call:
curl -v "https://blewit.us-west.host.bsky.network/xrpc/app.bsky.actor.getProfiles?$params" \
        -H "authorization: Bearer $accessJwt" \
        -H 'accept: application/json' -o "$tmp_body" 2> "$tmp_headers"

Expected behavior

Calls that don't specify accept-identity: gzip will not return gzipped data.

Details

  • Operating system: Ubuntu 22.04

Additional context

Calling the following returns plaintext.

curl -v "https://blewit.us-west.host.bsky.network/xrpc/app.bsky.actor.getProfiles?$params" \
        -H "authorization: Bearer $accessJwt" \
        -H 'accept-encoding: identity' -o "$tmp_body" 2> "$tmp_headers"

@scarnecchia scarnecchia added the bug Something isn't working label Sep 25, 2024
@scarnecchia scarnecchia changed the title ATProto/api — getActorProfiles suddenly defaulting to gzip output ATProto/api — actorGetProfiles suddenly defaulting to gzip output Sep 25, 2024
@scarnecchia scarnecchia changed the title ATProto/api — actorGetProfiles suddenly defaulting to gzip output ATProto/api — actor.GetProfiles suddenly defaulting to gzip output Sep 25, 2024
@scarnecchia
Copy link
Author

scarnecchia commented Sep 25, 2024

Possibly a feature resulting from optimizations in #2770 but will let the dev team decide if they want to close based on this: https://bsky.app/profile/dholms.xyz/post/3l4yrpshkhc2c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant