Flask-login current_user returns only ID

Question

I've managed to get flask-login working but when I try use, for example current_user.username in my templates the value is None

I suspect this is to do with my User.get() function but cannot for the life of my understand what I need to do so I can access the current users username or email and so on.

So the question is, why are the rest of my current_user attributes like username, not getting populated?

models.py

Note the get function

from sqlalchemy import Column, Integer, String
from flask_login import UserMixin

from database import Base


class User(Base, UserMixin):
    __tablename__ = 'users'
    id = Column(Integer, primary_key=True)
    username = Column(String(80), unique=True)
    password = Column(String(80))
    email = Column(String(120), unique=True)

    def __init__(self, id):
        self.id = id

    def __repr__(self):
        return '<User %r>' % self.username

    @classmethod
    def get(cls, id):
        """Return user instance of id, return None if not exist"""
        try:
            return cls(id)
        except UserWarning:
            return None

views.py

Note the load_user, I believe the flask documents expect this to be present and for it to take an ID as input

import flask
from flask_login import login_user, logout_user, login_required

from reportr import app, login_manager
from models import User
from forms import UploadForm, LoginForm
from database import init_session


@login_manager.user_loader
def load_user(id):
    return User.get(id)


@app.route('/', methods=('GET', 'POST'))
def index():
    return flask.render_template('index.html')


@app.route('/login', methods=['GET', 'POST'])
def login():
    form = LoginForm()
    del form.email  # Don't want the email address on the form

    if form.validate_on_submit():
        session = init_session()
        user = session.query(User).filter(User.username == flask.request.form['username']).first()

        if user and user.password == flask.request.form['password']:
            login_user(user)
            flask.flash('Logged in successfully.', 'info')
        else:
            flask.flash('Username or password incorrect.', 'error')

            return flask.redirect(flask.url_for('login'))

        return flask.redirect(flask.url_for('index'))
    return flask.render_template('login.html', form=form)


@app.route('/logout')
def logout():
    logout_user()

    return flask.redirect(flask.url_for('index'))


@app.route('/upload/results', methods=('GET', 'POST'))
def upload_results():
    form = UploadForm()

    if not form.validate_on_submit():
            flask.flash('Please fill out all required fields', 'error')

    return flask.render_template('upload_results.html', form=form)


@app.route("/settings")
@login_required
def settings():
    pass

templates/nav-bar.html

In here I am using current_user.id, this works.

{% block navbar %}
<nav class="navbar navbar-default" role="navigation">
    <div class="container">
        <div class="navbar-header">
            <a class="navbar-brand" href="{{ url_for('.index') }}"><span class="glyphicon-play" aria-hidden="true"></span>Reportr!</a>
        </div>
        <div class="navbar-collapse collapse click-nav">
            <ul class="nav navbar-nav no-js">
                <li class="dropdown">
                    <a id='browse_menu' href="#" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">Browse <span class="caret"></span></a>
                    <ul class="dropdown-menu" role="menu">
                        <li>
                            <a id='browse_by_project' href="#">By Project</a>
                        </li>
                    </ul>
                <li class="dropdown">
                    <a id='upload_menu' href="#" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">Upload <span class="caret"></span></a>
                    <ul class="dropdown-menu" role="menu">
                    <li>
                        <a id='upload_test_results' href="{{ url_for('.upload_results') }}">Test Results</a>
                    </li>
                    </ul>
                </li>
            </ul>
            <ul class="nav navbar-nav navbar-right">
                {% if current_user.is_authenticated() %}
                <li class="dropdown">
                <a href="#" class="dropdown-toggle" data-toggle="dropdown" role="button" aria-expanded="false">{{ current_user.id }} <span class="caret"></span></a>
                    <ul class="dropdown-menu" role="menu">
                        <li><a href="{{ url_for('.settings') }}">Settings</a></li>
                        <li class="divider"></li>
                        <li><a href="{{ url_for('.logout') }}">Logout</a></li>
                    </ul>
                </li>
                {% else %}
                <li>
                    <a id='login' href="{{ url_for('.login') }}">Login</a>
                </li>
                {% endif %}
            </ul>
      </div>
    </div>
</nav>
{% endblock %}

Answer 1

I've found the source of the issue.

My load_user() function was not returning a user, rather an instance of the user class with only the id populated.

I changed my load_user function to query the table and return the an instance of the user, instead the class and now it is working.

See below:

@login_manager.user_loader
def load_user(user_id):
    try:
        return User.query.get(user_id)
    except:
        return None

I also got rid of the get() function on my model as that is not needed and it is much simpler.