Fixed Handled error in OAuth2ExtraTokenMiddleware when authheader has Bearer with no token-string following up

[Tuhin-thinks]

Fixed the crash in application while using OAuth2ExtraTokenMiddleware. When Bearer token passed is empty.
Authorization: Bearer would result in this crash.

Fixes#1496

Description of the Change

Checklist

• PR only contains one change (considered splitting up PR)
• unit-test added
• documentation updated
• CHANGELOG.md updated (only for user relevant changes)
• author name in AUTHORS

[n2ygk]

Thanks for this fix. Please also add a test case in tests/test_auth_backends.py to assure full code coverage.

[dopry]

@Tuhin-thinks this looks good. I think the last task to get this merge ready is the test.

[Tuhin-thinks]

@dopry Thanks, I am getting them ready.

[dopry]

@Tuhin-thinks any updates on the tests?

[Tuhin-thinks]

@Tuhin-thinks any updates on the tests?

Hi, @dopry really sorry about the long wait.
Added test cases now, for middleware.py file's validation.

Please let me know once you find time to review this.

[dopry]

Hi, @dopry really sorry about the long wait. Added test cases now, for middleware.py file's validation.

Please let me know once you find time to review this.

Say nothing of it. It's open source and we're all volunteers. There isn't a schedule to keep here. :)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

His comments have been addressed. Tests are added and the split issue is resolved.