Skip to content

GroupUserMapper.scala: Backport fix from TheHive for group mapper functionality - #438

Merged
To-om merged 3 commits into
TheHive-Project:developfrom
jiprocha:fix-oauth2-group-mapper
Sep 20, 2023
Merged

GroupUserMapper.scala: Backport fix from TheHive for group mapper functionality#438
To-om merged 3 commits into
TheHive-Project:developfrom
jiprocha:fix-oauth2-group-mapper

Conversation

@jiprocha

@jiprocha jiprocha commented Dec 1, 2022

Copy link
Copy Markdown

This commit fixes issue #344 by backporting fix from TheHive repository. Original pull request from which the backport was taken can be found at TheHive-Project/TheHive#1112.

To-om and others added 3 commits June 22, 2022 17:57
…ctionality

This commit fixes issue TheHive-Project#344 by backporting fix from TheHive repository.
Original pull request from which the backport was taken can be found at
TheHive-Project/TheHive#1112.
@fkolacek

fkolacek commented Dec 1, 2022

Copy link
Copy Markdown

+1 for fixing this please

@JakubOnderka

Copy link
Copy Markdown

This fix will also solve our problem. @jiprocha Thanks!

@eviltik

eviltik commented Dec 14, 2022

Copy link
Copy Markdown

Does the version 3.1.7 include this fix ? ty

@jiprocha

Copy link
Copy Markdown
Author

@eviltik It does not, this is still pending to review, merge and release from the maintainers of Cortex. You can however clone 3.1.7 version of the repository, apply this patch and manually compile following the guide in documentation and it will work.

I've tried to reach out to the maintainers via email, without any response so far.

@nicocti

nicocti commented Mar 20, 2023

Copy link
Copy Markdown

Same here. Any news from the maintainers ?

@nadouani

Copy link
Copy Markdown
Contributor

Hi all, we are going to evaluate this feature for the upcoming release.

@To-om To-om added this to the 3.1.8 milestone Sep 19, 2023
@To-om
To-om changed the base branch from master to develop September 20, 2023 11:25
@To-om
To-om merged commit f7fdcdf into TheHive-Project:develop Sep 20, 2023
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.

7 participants