Catching bbolt panic and making it's use optionnal in links

bbriggs / bitbot

A golang IRC bot

MIT License

13 stars 15 forks source link

Catching bbolt panic and making it's use optionnal in links #213

Closed m-242 closed 3 years ago

m-242 commented 4 years ago

I couldn't test with conditions similar to deployment yet, so I have no idea if this works.

m-242 commented 4 years ago

Apart from the lack of testing, I'm still looking for a better solution to mark the accessibility of the embedded k/v db than having a boolean in the Bot struct.

m-242 commented 4 years ago

If the database fails to initialise, the bot's field value for it is nil, so we only need to test that the db isn't nil.

m-242 commented 3 years ago

Closing PR: we added the Init field for that in another PR.