Hypermodel `after_commit` handler is dropping errors instead of raising

Offen

#453 erstellt am 27.3.2023 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 35/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Veraltet
Tech-Stack: rails, ruby
Bereich: backend

Rechercherichtung

Beginne bei Hypermodels after_commit-Methode und untersuche das Ergebnis von SendPacket.run. Lies anschließend InternalPolicy#send_message, um zu verstehen, wie Fehler von Promises behandelt werden. Fertig ist die Änderung, wenn Fehler aus dem after-commit broadcast ausgelöst werden, statt verworfen zu werden, während das server-not-running rescue unverändert bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

in the after_commit method of hypermodel, the SendPacket operation is run which will return a promise, but it is not checked for an error state.

def self.after_commit(operation, model)
  # Calling public_columns_hash once insures all policies are loaded
  # before the first broadcast.
  @public_columns_hash ||= ActiveRecord::Base.public_columns_hash
  Hyperstack::InternalPolicy.regulate_broadcast(model) do |data|
    puts "Broadcast aftercommit hook: #{data}" if Hyperstack::Connection.show_diagnostics

    if !Hyperstack.on_server? && Hyperstack::Connection.root_path
      send_to_server(operation, data, model.__synchromesh_update_time) rescue nil # fails if server no longer running so ignore
    else
      SendPacket.run(data, operation: operation, updated_at: model.__synchromesh_update_time)
    end
  end
rescue ActiveRecord::StatementInvalid => e
  raise e unless e.message == "Could not find table 'hyperstack_connections'"
end unless RUBY_ENGINE == 'opal'

The line containing SendPacket should be

SendPacket.run(data, operation: operation, updated_at: model.__synchromesh_update_time).tap { |p| raise p.error if p.error }

The easiest way to patch this is by adding this:

module Hyperstack
  class InternalPolicy
    alias original_send_message send_message
    def send_message(*args, &block)
      original_send_message(*args, &block).tap { |p| raise p.error if p.is_a?(Promise) && p.error }
    end  
  end
end

This will catch the error with the smallest patch. Note the patch needs to check if its a promise, but the actual fix does not.

Vorherrschende Sprache: JavaScript
Sterne: 538
Forks: 41
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus hyperstack-org/hyperstack

expose_as_operations method Offen

enhancement needs doc

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

hyperstack-org/hyperstack#359 · Erstellt am 24.2.2021
Incorrect documentation about server operations Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100

hyperstack-org/hyperstack#267 · Erstellt am 07.10.2019
improve server_method and finder_method Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100

hyperstack-org/hyperstack#457 · Erstellt am 29.6.2023
./spec/batch4/default_value_spec.rb:40 failing on latest react Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100

hyperstack-org/hyperstack#456 · Erstellt am 01.6.2023
don't transmit full record on change - only send changed values Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100

hyperstack-org/hyperstack#455 · 2 Kommentare · Erstellt am 31.5.2023

Alle Issues in hyperstack-org/hyperstack