Nisaba / btcpayserver-plugins

MIT License
1 stars 0 forks source link

need enhanced messaging from Breez plug-in for BTCPay #2

Open dme1sc opened 1 month ago

dme1sc commented 1 month ago

Hi, what is needed to get Serilog plugin to send Breez plugin fo btcpay to send the breez logs to slack also? I found that the Breez plugin generates logs but are not inlcuded. Thanks! Love the logging to slack - works great.

Nisaba commented 1 month ago

Hi, Yes it seems that the Serilog plugin sends only logs from BTCPay core but not logs from other plugins...

I need to see that and publish an update if it's feasible

dme1sc commented 1 month ago

side note - your lightning address is coming up invalid... donate@btcpay.nisaba.solutions

Nisaba commented 1 month ago

yes, you're right ! I have to update this... You can try donate@btcpay2.nisaba.solutions or https://btcpay2.nisaba.solutions/apps/33A4iL9jhsbTsP1SHGTo8VHgmU4Y/pos

Nisaba commented 1 month ago

Hi, I don't use Breeze plugin but I tried with another plugin (B2P Central) and it works... Are you sure the Minimum information level set in Serilog plugin config is OK with the notification levels from Breeze logs ?

dme1sc commented 1 month ago

I don't - how would i check that?

Doug -- Doug Morton | @.*** | 843-936-0632 AlpacaRMS https://alpacarms.com - Retail Management System and Point of Sale

To contact the Helpdesk, please send an email to: @.*** from your main account email on file. Create a separate email per issue and put a brief summary of the issue in your subject. Check case status at: https://carolinapos.on.spiceworks.com

On Mon, Sep 16, 2024 at 3:07 PM Nisaba @.***> wrote:

Hi, I don't use Breeze plugin but I tried with another plugin (B2P Central) and it works... Are you sure the Minimum information level set in Serilog plugin config is OK with the notification levels from Breeze logs ?

— Reply to this email directly, view it on GitHub https://github.com/Nisaba/btcpayserver-plugins/issues/2#issuecomment-2353704739, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVJD56UH4KH73P7AMMW22VTZW4UAHAVCNFSM6AAAAABOHFV2NKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJTG4YDINZTHE . You are receiving this because you authored the thread.Message ID: @.***>

Nisaba commented 1 month ago

here image

And you compare with the level for the breeze notifications in the log file

Nisaba commented 1 month ago

Hi, Did you resolved your issue ?

dme1sc commented 1 month ago

No. It's a coding thing with the breez plugin. The developer and I discussed. He doesnt have the time to code the serilog levels for the messaging I want. I was going to get my dev to take a look but my dev isn't trained on the code. It's this sdk: https://sdk-doc-greenlight.breez.technology/guide/logging.html I'm basically just wanting to code the Channel Closed messages as Warnings and then catch them in your plugin to send to Slack.

Doug --

On Tue, Sep 17, 2024 at 7:05 PM Nisaba @.***> wrote:

Hi, Did you resolved your issue ?

— Reply to this email directly, view it on GitHub https://github.com/Nisaba/btcpayserver-plugins/issues/2#issuecomment-2357100175, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVJD56Q46FRMIHE5IV23XNTZXCYRXAVCNFSM6AAAAABOHFV2NKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJXGEYDAMJXGU . You are receiving this because you authored the thread.Message ID: @.***>

Nisaba commented 1 month ago

Yes, even in BTCPay core, event levels are not always well defined... Updating log levels is not difficult for the breez developper, only a couple of minutes for that...

But the problem I guess is the publication process againt other updates in their code...