Warning: Can't synchronize with repository "(default)" (No changeset 96d22ec3fa3ef6de3ea8dc0d7d398adc9aa071cf in the repository). Look in the Trac log for more information.

Ticket #541 (closed defect: fixed)

Opened 3 years ago

Last modified 4 months ago

oeventsd does case-sensitive comparisions for trigger statuses

Reported by: PaulFertser Owned by: jluebbe
Priority: minor Milestone:
Component: framework/oeventsd Version:
Keywords: Cc:

Description

This makes oeventsd config incompatible between ogsmd and fsogsmd because the former is using all-lowercase and the latter all-uppercase statuses (e.g. INCOMING).

Change History

comment:1 Changed 3 years ago by mickey

  • Status changed from new to in_testing

commit 4e20053601d1f1d108ed959b4a62636c20e0a34a Author: Michael 'Mickey' Lauer <mickey@…> Date: Thu Apr 15 17:24:40 2010 +0200

ophoned: use lower case comparison for call status

This should fix interoperability with fsogsmd

comment:2 Changed 3 years ago by PaulFertser

  • Status changed from in_testing to closed
  • Resolution set to fixed

Big thanks, it's fixed.

comment:3 Changed 3 years ago by PaulFertser

  • Status changed from closed to reopened
  • Resolution fixed deleted

Hm, probably i should sleep more, i thought i checked it but then realised i do not get triggers, looks to me like you use lower() for the wrong side of == :) I'd convert to lowercase the incoming status immediately upon receiving.

comment:4 Changed 3 years ago by mickey

Which file & line do you mean?

comment:5 Changed 4 months ago by PaulFertser

  • Status changed from reopened to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.