Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Always return code 200 in session verification endpoint #4622

Merged
merged 1 commit into from
Oct 16, 2020
Merged

Conversation

ikapelyukhin
Copy link
Contributor

Description

Motivation and Context

#4533

How Has This Been Tested?

Autotests and manually.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Docs update
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have followed the guidelines in CONTRIBUTING.md, including the required formatting of the commit message

@codecov-commenter
Copy link

Codecov Report

Merging #4622 into master will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #4622   +/-   ##
=======================================
  Coverage   58.41%   58.41%           
=======================================
  Files         883      883           
  Lines       29301    29301           
  Branches     4253     4253           
=======================================
  Hits        17115    17115           
  Misses      12186    12186           

Copy link
Contributor

@nwmac nwmac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nwmac nwmac merged commit ce4c108 into master Oct 16, 2020
@nwmac nwmac deleted the checkVerify200 branch October 16, 2020 13:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants