Is site.com/club/their-name-here
a public page? It seems if they are not logged in and redirect to /club/
which shows a login then it is only visible to users logged in? Why not create a universal login page at example.com/login
and once authenticated redirect to site.com/club/their-club-name
Or for that matter site.com/their-name
The redirect you have isn't bad since /club/
most likely doesn't have any content, so 301 redirecting isn't taking away any link juice to a page with any value, so there isn't much to worry about in that regard.
Is /club
really needed? Or is it there because of how the website and application were developed? Would your website still make sense to visitors and users if it were just site.com/their-name
?