gorilla / csrf

Is there an existing issue for this?

I have searched the existing issues

Current Behavior

Based on my understanding and testing, it appears that the variable should actually contain Host values (e.g., github.com) rather than full Origin values (e.g., https://github.com).

csrf/csrf.go

Lines 261 to 262 in a009743

    
           for _, trustedOrigin := range cs.opts.TrustedOrigins { 
        
           	if referer.Host == trustedOrigin {

This distinction was not immediately clear from the documentation or the code comments (This is stated in the only README).

Expected Behavior

To avoid potential confusion, would it be possible to consider renaming the variable to something more indicative of its intended content, such as TrustedHosts?

If renaming is not feasible for backward compatibility, perhaps adding a more explicit explanation or comment in the code to clarify the expected format of values could be helpful.

csrf/options.go

Lines 123 to 128 in a009743

    
           // TrustedOrigins configures a set of origins (Referers) that are considered as trusted. 
        
           // This will allow cross-domain CSRF use-cases - e.g. where the front-end is served 
        
           // from a different domain than the API server - to correctly pass a CSRF check. 
        
           // 
        
           // You should only provide origins you own or have full control over. 
        
           func TrustedOrigins(origins []string) Option {

Steps To Reproduce

No response

Anything else?