logoalt Hacker News

fastest963last Wednesday at 3:29 AM2 repliesview on HN

I'm not sure there's a realistic alternative. If you need to generate a key then it has to happen somehow on unsupported platforms. You can check Enabled() if you need to know and intend to do something different but I assume most of the time you run the same function either way you'd just prefer to opt into secret mode if it's available.


Replies

kbolinoyesterday at 6:53 PM

This is not what secret.Enabled() means. But it probably illustrates that the function needs to be renamed already. Here's what the doc comment says:

  // Enabled reports whether Do appears anywhere on the call stack.
In other words, it is just a way of checking that you are indeed running inside the context of some secret.Do call; it doesn't guarantee that secret.Do is actually offering the protection you may desire.
show 2 replies
awithrowlast Wednesday at 5:04 AM

Why not just panic and make it obvious?

show 1 reply