Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use a couple of defer in internal/home/auth.go #2471

Merged
merged 1 commit into from
Dec 23, 2020
Merged

Use a couple of defer in internal/home/auth.go #2471

merged 1 commit into from
Dec 23, 2020

Conversation

jvoisin
Copy link
Contributor

@jvoisin jvoisin commented Dec 21, 2020

No description provided.

@ameshkov ameshkov requested a review from ainar-g December 22, 2020 11:35
@ainar-g
Copy link
Contributor

ainar-g commented Dec 22, 2020

Thanks, we'll review this shortly.

adguard pushed a commit that referenced this pull request Dec 23, 2020
@ainar-g
Pull request: 2471 defer
e829e7a 
Merge in DNS/adguard-home from 2471-defer to master

Updates #2471.

* commit '1c754788f9139ed9741cf01c6d94bcced6909b8c':
  home: improve getCurrentUser
  home: improve checkSession
  Use a couple of defer in internal/home/auth.go
@adguard adguard merged commit 7eb3e00 into AdguardTeam:master Dec 23, 2020
@ainar-g ainar-g added this to the v0.105.0 milestone Dec 23, 2020
@ainar-g ainar-g self-assigned this Dec 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ainar-g ainar-g Awaiting requested review from ainar-g

Assignees

@ainar-g ainar-g

Labels
enhancement
Projects
None yet
Milestone
v0.105.0
Development

Successfully merging this pull request may close these issues.
3 participants
@jvoisin @ainar-g @adguard