sebb wrote on 1/24/25 6:39 PM:
On Fri, 24 Jan 2025 at 20:01, <curc...@apache.org> wrote:
...snip...
+ # @return hash, or string error if not found
def self.get_timeline(mtg_root)
begin
return JSON.parse(IO.read(File.join(mtg_root, 'runbook',
'timeline.json')))
rescue StandardError => e
- return null
+ return "ERROR: get_timeline(#{mtg_root}) threw: #{e.message}"
I don't see any checks on the return type by callers of this routine.
AFAICT they all assume it is a hash.
Correct - I still don't have a working dev environment so have been
working on the server, which is fine since these tools are not yet
actively being used. So I'm experimenting as I cleanup methods between
the two nominate scripts, as well as adding the workflow features I'm
proposing we use for the next go-round.
My concept was that if the caller cares, check the return value; if not
a hash, then use the returned string as a specific error that provides
additional debugging information.
Sadly, my --docker attempts fail on ldap, and my non-docker attempts
fail with a "AH06665: No code signing... for mod_passenger.so" Which is
odd, because homebrew says that they now do sign their modules...
Wondering if I'm just cursed with bad configs this week, or if this is
something to do with Sequoia 15.2 on Apple Silicon M4 chip.
--
- Shane
Member
The Apache Software Foundation