Skip to content

Commit

Permalink
chore: Set X-User-Id in calls to our backend
Browse files Browse the repository at this point in the history
Anonymous id potentially useful for error correlation
  • Loading branch information
Amund211 committed Mar 22, 2024
1 parent 3c74976 commit 2088eec
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
5 changes: 4 additions & 1 deletion src/prism/overlay/antisniper_api.py
Original file line number Diff line number Diff line change
Expand Up @@ -135,14 +135,15 @@ def _make_request(
def _make_playerdata_request(
*,
url: str,
user_id: str,
key_holder: AntiSniperAPIKeyHolder,
api_limiter: RateLimiter,
last_try: bool,
) -> requests.Response: # pragma: nocover
try:
# Uphold our prescribed rate-limits
with key_holder.limiter, api_limiter:
response = SESSION.get(url)
response = SESSION.get(url, headers={"X-User-Id": user_id})
except RequestException as e:
raise ExecutionError(
"Request to AntiSniper API failed due to an unknown error"
Expand All @@ -161,6 +162,7 @@ def _make_playerdata_request(

def get_antisniper_playerdata(
uuid: str,
user_id: str,
key_holder: AntiSniperAPIKeyHolder,
api_limiter: RateLimiter,
retry_limit: int = 5,
Expand All @@ -175,6 +177,7 @@ def get_antisniper_playerdata(
functools.partial(
_make_playerdata_request,
url=url,
user_id=user_id,
key_holder=key_holder,
api_limiter=api_limiter,
),
Expand Down
5 changes: 4 additions & 1 deletion src/prism/overlay/controller.py
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,10 @@ def get_antisniper_playerdata(
# TODO: set api key flags
try:
playerdata = get_antisniper_playerdata(
uuid, self.antisniper_key_holder, self.api_limiter
uuid,
self.settings.user_id,
self.antisniper_key_holder,
self.api_limiter,
)
except HypixelPlayerNotFoundError as e:
logger.debug(f"Player not found on Hypixel: {uuid=}", exc_info=e)
Expand Down

0 comments on commit 2088eec

Please sign in to comment.