Refactor DebuginfodServer initialization error handling

[mysterymath]

Currently, DebuginfodServer crashes the program in its constructor if the HTTP Server cannot be set up (i.e., if some aspect of setting up the wrapped HTTPServer instance fails). This responsibility more properly falls on the program instantiating the DebuginfodServer instance. Accordingly, this should be changed to use a factory method, so that the instantiating program can decide how to handle failure.

llvm-project/llvm/lib/Debuginfod/Debuginfod.cpp

Line 518 in 4df01dc

DebuginfodServer::DebuginfodServer(DebuginfodLog &Log,

[markram1729]

Hey , is this issue resolved ?
can i work on this ?

[mysterymath]

Yes, this issue is still available.

[markram1729]

Hey @mysterymath , my apologies for delay

can you please share how can i reproduce this issue, I am pretty new to llvm project would like to have your suggestions

[mysterymath]

It's not exactly an bug, more a code cleanliness thing. The cantFail lines preclude a user of the DebuginfodServer class handling the error; the task is to move construction out into a static factory method that returns the Error so that the caller can report it gracefully.

[markram1729]

oh! got it
I will start working on it

[markram1729]

Hi @mysterymath
MY apologies for the delay
I have cleaned code accordingly and added factory method

can you please check it