Add "MFA Verified" check to workflowengine #40235
Conversation
Signed-off-by: MohammadReza vahedi <mr.vahedi68@gmail.com>
Signed-off-by: Michiel de Jong <michiel@pondersource.com>
michielbdejong
force-pushed
the
mrv/mfa-check-rebased
branch
from
ee9c992
to
02604be
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
apps/workflowengine/lib/Manager.php
Outdated
| @@ -759,6 +767,7 @@ | |||
| $this->container->query(FileName::class), | |||
| $this->container->query(FileSize::class), | |||
| $this->container->query(FileSystemTags::class), | |||
| $this->container->query(MfaVerified::class), | |||
Check notice
Code scanning / Psalm
DeprecatedMethod Note
Signed-off-by: MohammadReza vahedi <mr.vahedi68@gmail.com>
|
Can this be merged if it looks good? |
|
@come-nc what is the next step in the process now? |
come-nc
requested review from
a team,
ArtificialOwl,
icewind1991 and
Fenn-CS
and removed request for
a team
|
@ArtificialOwl can you maybe have a look? |
|
I'm worried about the lack of communication here, @mickenordin can you ask your NC support contact for advice? |
|
@skjnldsv Thanks for mentioning this PR for NC 28! Will you be the person merging it? Anything we can do to help you? Note that the conflicting files in the dist folders are easy to leave out and then rebuild after you merge! |
|
@michielbdejong you're missing two reviews. Pinging maintainers nonetheless: @blizzz and @juliushaertl |
|
With beta from last week, we're behind feature freeze for 28 😧 |
| * @copyright Copyright (c) 2016 Joas Schilling <coding@schilljs.com> | ||
| * | ||
| * @author Arthur Schiwon <blizzz@arthur-schiwon.de> | ||
| * @author Christoph Wurst <christoph@winzerhof-wurst.at> | ||
| * @author Joas Schilling <coding@schilljs.com> | ||
| * @author Julius Härtl <jus@bitgrid.net> | ||
| * @author Richard Steinmetz <richard@steinmetz.cloud> |
Signed-off-by: Michiel de Jong <michiel@pondersource.com>
Co-authored-by: Arthur Schiwon <blizzz@arthur-schiwon.de> Signed-off-by: Michiel de Jong <michiel@unhosted.org>
|
I succeeded in adding this check inside https://github.com/pondersource/mfazones instead of adding it to nextcloud/server, so we can close this PR. If anyone wants to use the |
Rebase of #39471