Skip to content

Commit

Permalink
SigHelper: Fix some logic errors raised during code review
Browse files Browse the repository at this point in the history
  • Loading branch information
SamantazFox committed Aug 7, 2024
1 parent 7798faf commit cc36a82
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/invidious/helpers/sig_helper.cr
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ module Invidious::SigHelper

# Return when the player was last updated
def get_player_timestamp : UInt64?
request = Request.new(Opcode::GET_SIGNATURE_TIMESTAMP, nil)
request = Request.new(Opcode::PLAYER_UPDATE_TIMESTAMP, nil)

return self.send_request(request) do |bytes|
IO::ByteFormat::NetworkEndian.decode(UInt64, bytes)
Expand Down
8 changes: 4 additions & 4 deletions src/invidious/helpers/signatures.cr
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@ class Invidious::DecryptFunction
# If we have updated in the last 5 minutes, do nothing
return if (now - @last_update) > 5.minutes

# Get the time when the player was updated, in the event where
# multiple invidious processes are run in parallel.
player_time = Time.unix(@client.get_player_timestamp || 0)
# Get the amount of time elapsed since when the player was updated, in the
# event where multiple invidious processes are run in parallel.
update_time_elapsed = (@client.get_player_timestamp || 301).seconds

if (now - player_time) > 5.minutes
if update_time_elapsed > 5.minutes
LOGGER.debug("Signature: Player might be outdated, updating")
@client.force_update
@last_update = Time.utc
Expand Down

0 comments on commit cc36a82

Please sign in to comment.