Skip to content

VIVO-4178 Fix for report generation authorization#528

Open
litvinovg wants to merge 1 commit into
vivo-project:mainfrom
litvinovg:4178
Open

VIVO-4178 Fix for report generation authorization#528
litvinovg wants to merge 1 commit into
vivo-project:mainfrom
litvinovg:4178

Conversation

@litvinovg

Copy link
Copy Markdown
Member

VIVO GitHub issue

  • Other Relevant Links (Mailing list discussion, related pull requests, etc.)

What does this pull request do?

  • Fixes authorization request
  • Redirects user to login page in case authorization fails

How should this be tested?

  • Reproduce the problem as described in the linked issue
  • Test that the pull request fixes authorization and redirects user to the login page.

Interested parties

@VIVO-project/vivo-committers

Reviewers' expertise

Candidates for reviewing this PR should have some of the following expertises:

  1. Java

Reviewers' report template

General comment

A reviewer should provide here comments and suggestions for requested changes if any.

Testing

A reviewer should briefly describe here how it was tested

Code reviewing

A reviewer should briefly describe here which part was code reviewed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Report generation continues despite failed authorization check

1 participant