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.

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

Introduce AUTH_REQUIRE_TOKENS #3007

Merged
merged 3 commits into from
Oct 27, 2023
Merged

Introduce AUTH_REQUIRE_TOKENS #3007

merged 3 commits into from
Oct 27, 2023

Conversation

nextgens
Copy link
Contributor

What type of PR?

enhancement

What does this PR do?

Introduce AUTH_REQUIRE_TOKENS to enforce that thick clients use tokens instead of passwords

Related issue(s)

Prerequisites

Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

  • In case of feature or enhancement: documentation updated accordingly
  • Unless it's docs or a minor change: add changelog entry file.

@nextgens nextgens added priority/p2 Minor bug / Could have type/security Related to security type/enhancement Enhances existing functionality labels Oct 27, 2023
@mergify
Copy link
Contributor

mergify bot commented Oct 27, 2023

Thanks for submitting this pull request.
Bors-ng will now build test images. When it succeeds, we will continue to review and test your PR.

bors try

Note: if this build fails, read this.

bors bot added a commit that referenced this pull request Oct 27, 2023
@bors
Copy link
Contributor

bors bot commented Oct 27, 2023

try

Build failed:

@nextgens nextgens requested a review from ghostwheel42 October 27, 2023 13:15
Copy link
Contributor

@ghostwheel42 ghostwheel42 left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify
Copy link
Contributor

mergify bot commented Oct 27, 2023

bors r+

bors bot added a commit that referenced this pull request Oct 27, 2023
3007: Introduce AUTH_REQUIRE_TOKENS r=mergify[bot] a=nextgens

## What type of PR?

enhancement

## What does this PR do?

Introduce AUTH_REQUIRE_TOKENS to enforce that thick clients use tokens instead of passwords

### Related issue(s)

## Prerequisites
Before we can consider review and merge, please make sure the following list is done and checked.
If an entry in not applicable, you can check it or remove it from the list.

- [ ] In case of feature or enhancement: documentation updated accordingly
- [x] Unless it's docs or a minor change: add [changelog](https://mailu.io/master/contributors/workflow.html#changelog) entry file.


Co-authored-by: Florent Daigniere <nextgens@freenetproject.org>
@bors
Copy link
Contributor

bors bot commented Oct 27, 2023

Build failed:

@nextgens
Copy link
Contributor Author

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 27, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 26e5618 into Mailu:master Oct 27, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority/p2 Minor bug / Could have type/enhancement Enhances existing functionality type/security Related to security
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants