<p>hey everyone! first time poster in this subreddit. I've been working with go for about 4 months now and absolutely love it.</p>
<p>Anyways, I'm working on a relatively complex web application back-end and have gotten a little concerned that in places where I need to return errors, I am using string literals for the string value which I'm starting to think could be a major headache later:</p>
<pre><code>if roles.IsAdmin(&user) != true {
return fmt.Errorf("UNAUTHORIZED")
}
</code></pre>
<p>So I'm wondering: would it be bad practice to define in my application an <code>errors</code> module and define every application error string in there? E.g:</p>
<pre><code>import "github.com/me/myproject/errors"
if roles.IsAdmin(&user) != true {
return fmt.Errorf(errors.ErrUnauthorized)
}
</code></pre>
<p>Or is the correct approach to have the errors defined in the module of which they could be produced, e.g:</p>
<pre><code>if roles.IsAdmin(&user) != true {
return fmt.Errorf(roles.ErrUnauthorized)
}
</code></pre>
