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

Fix integration test failure by allowing direct access to system index warning #784

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

gaobinlong
Copy link
Collaborator

@gaobinlong gaobinlong commented Oct 11, 2023

Description

Fix the integration test failure of org.opensearch.integtest.config.ChimeNotificationConfigCrudIT which throws org.opensearch.client.WarningFailureException when refreshing all the indices in the test cluster, the change of this PR is that only refresh the system index .opensearch-notification-config in this plugin and allow the warning when refreshing the index.

Issues Resolved

#783

Check List

  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…x warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Hailong-am
Hailong-am previously approved these changes Oct 11, 2023
xluo-aws
xluo-aws previously approved these changes Oct 11, 2023
@Hailong-am
Copy link
Collaborator

bwc will not pass on main until we bump the bwc version to 2.12

@Hailong-am
Copy link
Collaborator

test the changes on my local rpm package with command ./gradlew integTest -Dtests.rest.cluster="localhost:9200" -Dtests.cluster="localhost:9200" -Dtests.clustername="opensearch-integrationtest" -Dhttps=false, it works well.

…etting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>
@gaobinlong gaobinlong dismissed stale reviews from xluo-aws and Hailong-am via 415148a October 11, 2023 10:03
@gaobinlong gaobinlong merged commit 0a7a5f9 into opensearch-project:main Oct 11, 2023
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/notifications/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/notifications/backport-2.x
# Create a new branch
git switch --create backport/backport-784-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 0a7a5f99e71248cce9d5c15b0b99f9f270261d21
# Push it to GitHub
git push --set-upstream origin backport/backport-784-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/notifications/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-784-to-2.x.

@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.11 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/notifications/backport-2.11 2.11
# Navigate to the new working tree
pushd ../.worktrees/notifications/backport-2.11
# Create a new branch
git switch --create backport/backport-784-to-2.11
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 0a7a5f99e71248cce9d5c15b0b99f9f270261d21
# Push it to GitHub
git push --set-upstream origin backport/backport-784-to-2.11
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/notifications/backport-2.11

Then, create a pull request where the base branch is 2.11 and the compare/head branch is backport/backport-784-to-2.11.

gaobinlong added a commit to gaobinlong/notifications that referenced this pull request Oct 11, 2023
…x warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
(cherry picked from commit 0a7a5f9)
gaobinlong added a commit to gaobinlong/notifications that referenced this pull request Oct 11, 2023
…x warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
(cherry picked from commit 0a7a5f9)
xluo-aws pushed a commit that referenced this pull request Oct 11, 2023
…x warning (#784) (#785)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
(cherry picked from commit 0a7a5f9)
Hailong-am pushed a commit that referenced this pull request Oct 12, 2023
…x warning (#784) (#786)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
(cherry picked from commit 0a7a5f9)
rachana-dani pushed a commit to rachana-dani/notifications that referenced this pull request Oct 19, 2023
…x warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
zhichao-aws pushed a commit that referenced this pull request Oct 23, 2023
* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 23, 2023
* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
(cherry picked from commit 76ddcd4)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
zhichao-aws pushed a commit that referenced this pull request Oct 24, 2023
* Replace the TestMailServer to GreenMail server (#801)

* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
(cherry picked from commit 76ddcd4)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Add slf4j-api for greenmail

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Noir01 pushed a commit to Noir01/notifications that referenced this pull request Nov 13, 2023
…x warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Noir01 added a commit to Noir01/notifications that referenced this pull request Nov 13, 2023
* Add 2.11 release notes (opensearch-project#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (opensearch-project#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (opensearch-project#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (opensearch-project#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Noir01 pushed a commit to Noir01/notifications that referenced this pull request Nov 16, 2023
…x warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>
Noir01 added a commit to Noir01/notifications that referenced this pull request Nov 16, 2023
* Add 2.11 release notes (opensearch-project#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (opensearch-project#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (opensearch-project#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (opensearch-project#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (opensearch-project#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>
zhichao-aws added a commit that referenced this pull request Nov 20, 2023
* Microsoft teams (#676)

* Added feature support for microsoft teams webhoo

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* Added feature support for microsoft teams webhook ,removed valid webhooks

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* Added feature support for Microsoft teams webhook

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* Refactored feature support for  ms teams and added unit and integTest

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* fix build in core

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix core-spi build

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix notifications main code

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix mappings, add IT

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add auto upgrade mapping logic

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* put load mapping to initialize step

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add schema_version field

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add integ test

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* adjust with auto upgrade mapping logic

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add bwc

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* modify bwc

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* modify bwc

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* resolve  comments

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add license header

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix microsoft teams sample url in IT to adapt url validation

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

---------

Signed-off-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Co-authored-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add microsoft teams validation error message (#746)

* add validation failure message for Microsoft Teams

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* modify integtest

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

---------

Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* onboard system and hidden index (#742)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Updates demo certs used in integ tests (#756)

Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add 2.10.0 release notes (#755)

* Add 2.10.0 release notes

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* update release notes

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* Update opensearch-notifications.release-notes-2.10.0.0.md

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* fix wrong PR number

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* bump bwc version to 2.11 (#763)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Re-enable detekt

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Impove security plugin enabling check (#792)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add github workflow to auto bump bwc version  (#799)

* Adding bump bwc version github workflow

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* revert app id

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replace the TestMailServer to GreenMail server (#801)

* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Onboard prod jenkins docker image to github actions (#809)

* Onboard prod jenkins docker image to github actions

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Add more

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

---------

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Added org.apache.logging.log4j:log4j-slf4j-impl to classpath (#791)

* Added slf4j-jdk14.jar to classpath

Adding binding for SLF4J that should fix StaticLoggerBinder being not loaded.
Followed instructions from warning messages that appear.

Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>

* Undid 80fc198

Removed slf4j-jdk14.jar from classpath

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>

* Added org.apache.logging.log4j:log4j-slf4j-impl to classpath

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>

---------

Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Added Slack webhook URL validation regex

Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replaced wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh Srivastava <ansriv2005@gmail.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replaced more wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replaced even more wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replace path of mock Slack URL with `sample_slack_url`

Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Remove slackId from domainIds

Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replace wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add tests for wrong Slack URLs

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add validation tests for Slack URL

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Format

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* GovSlack apps can use the slack-gov.com domain

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add validation for gov-slack.com domain

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

---------

Signed-off-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <ansriv2005@gmail.com>
Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Co-authored-by: zhichao-aws <zhichaog@amazon.com>
Co-authored-by: danielkyalo599 <johbiento@gmail.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Darshit Chanpura <35282393+DarshitChanpura@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: Rachana Dani <36135368+rachana-dani@users.noreply.github.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 23, 2024
* Microsoft teams (#676)

* Added feature support for microsoft teams webhoo

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* Added feature support for microsoft teams webhook ,removed valid webhooks

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* Added feature support for Microsoft teams webhook

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* Refactored feature support for  ms teams and added unit and integTest

Signed-off-by: danielkyalo599 <johbiento@gmail.com>

* fix build in core

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix core-spi build

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix notifications main code

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix mappings, add IT

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add auto upgrade mapping logic

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* put load mapping to initialize step

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add schema_version field

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add integ test

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* adjust with auto upgrade mapping logic

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add bwc

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* modify bwc

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* modify bwc

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* resolve  comments

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* add license header

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* fix microsoft teams sample url in IT to adapt url validation

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

---------

Signed-off-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Co-authored-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add microsoft teams validation error message (#746)

* add validation failure message for Microsoft Teams

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

* modify integtest

Signed-off-by: zhichao-aws <zhichaog@amazon.com>

---------

Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* onboard system and hidden index (#742)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Updates demo certs used in integ tests (#756)

Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add 2.10.0 release notes (#755)

* Add 2.10.0 release notes

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* update release notes

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* Update opensearch-notifications.release-notes-2.10.0.0.md

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* fix wrong PR number

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* bump bwc version to 2.11 (#763)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Re-enable detekt

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Impove security plugin enabling check (#792)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add github workflow to auto bump bwc version  (#799)

* Adding bump bwc version github workflow

Signed-off-by: Hailong Cui <ihailong@amazon.com>

* revert app id

Signed-off-by: Hailong Cui <ihailong@amazon.com>

---------

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replace the TestMailServer to GreenMail server (#801)

* Add 2.11 release notes (#774)

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning

Signed-off-by: gaobinlong <gbinlong@amazon.com>

* Fix bwc test failure of throwing direct access to system index when getting mapping

Signed-off-by: gaobinlong <gbinlong@amazon.com>

---------

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Replace the TestMailServer to GreenMail server

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* bump bwc version to 2.12 (#793)

Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update dependency org.json:json to v20231013 (#795)

Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Re-enable detekt (#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Add assertion for retrieval of notification

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to stable version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

* Update to suggested version

Signed-off-by: rdani <rachana_sudhindradani@intuit.com>

---------

Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Onboard prod jenkins docker image to github actions (#809)

* Onboard prod jenkins docker image to github actions

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

* Add more

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>

---------

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Added org.apache.logging.log4j:log4j-slf4j-impl to classpath (#791)

* Added slf4j-jdk14.jar to classpath

Adding binding for SLF4J that should fix StaticLoggerBinder being not loaded.
Followed instructions from warning messages that appear.

Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>

* Undid 80fc198

Removed slf4j-jdk14.jar from classpath

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>

* Added org.apache.logging.log4j:log4j-slf4j-impl to classpath

Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>

---------

Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Added Slack webhook URL validation regex

Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replaced wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh Srivastava <ansriv2005@gmail.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replaced more wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replaced even more wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replace path of mock Slack URL with `sample_slack_url`

Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Remove slackId from domainIds

Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Replace wrongly formatted dummy Slack URL with properly formatted dummy Slack URL

Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add tests for wrong Slack URLs

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add validation tests for Slack URL

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Format

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* GovSlack apps can use the slack-gov.com domain

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

* Add validation for gov-slack.com domain

Signed-off-by: Aniruddh <ansriv2005@gmail.com>

---------

Signed-off-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <ansriv2005@gmail.com>
Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Co-authored-by: zhichao-aws <zhichaog@amazon.com>
Co-authored-by: danielkyalo599 <johbiento@gmail.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Darshit Chanpura <35282393+DarshitChanpura@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: Rachana Dani <36135368+rachana-dani@users.noreply.github.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
(cherry picked from commit 87109a0)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
amsiglan pushed a commit that referenced this pull request Aug 23, 2024
* Microsoft teams (#676)

* Added feature support for microsoft teams webhoo



* Added feature support for microsoft teams webhook ,removed valid webhooks



* Added feature support for Microsoft teams webhook



* Refactored feature support for  ms teams and added unit and integTest



* fix build in core



* fix core-spi build



* fix notifications main code



* fix mappings, add IT



* add auto upgrade mapping logic



* put load mapping to initialize step



* add schema_version field



* add integ test



* adjust with auto upgrade mapping logic



* add bwc



* modify bwc



* modify bwc



* resolve  comments



* add license header



* fix microsoft teams sample url in IT to adapt url validation



---------






* Add microsoft teams validation error message (#746)

* add validation failure message for Microsoft Teams



* modify integtest



---------




* onboard system and hidden index (#742)




* Updates demo certs used in integ tests (#756)




* Add 2.10.0 release notes (#755)

* Add 2.10.0 release notes



* update release notes



* Update opensearch-notifications.release-notes-2.10.0.0.md



* fix wrong PR number



---------




* bump bwc version to 2.11 (#763)




* Add 2.11 release notes (#774)




* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning



* Fix bwc test failure of throwing direct access to system index when getting mapping



---------





* Re-enable detekt

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`





* bump bwc version to 2.12 (#793)





* Update dependency org.json:json to v20231013 (#795)





* Impove security plugin enabling check (#792)





* Add github workflow to auto bump bwc version  (#799)

* Adding bump bwc version github workflow



* revert app id



---------





* Replace the TestMailServer to GreenMail server (#801)

* Add 2.11 release notes (#774)




* Fix integration test failure by allowing direct access to system index warning (#784)

* Fix integration test failure by allowing direct access to system index warning



* Fix bwc test failure of throwing direct access to system index when getting mapping



---------




* Replace the TestMailServer to GreenMail server



* bump bwc version to 2.12 (#793)




* Update dependency org.json:json to v20231013 (#795)




* Re-enable detekt (#796)

Bumped version of `io.gitlab.arturbosch.detekt:detekt-gradle-plugin` to `1.23.0`





* Add assertion for retrieval of notification



* Update to stable version



* Update to stable version



* Update to suggested version



---------














* Onboard prod jenkins docker image to github actions (#809)

* Onboard prod jenkins docker image to github actions



* Add more



---------





* Added org.apache.logging.log4j:log4j-slf4j-impl to classpath (#791)

* Added slf4j-jdk14.jar to classpath

Adding binding for SLF4J that should fix StaticLoggerBinder being not loaded.
Followed instructions from warning messages that appear.



* Undid 80fc198

Removed slf4j-jdk14.jar from classpath



* Added org.apache.logging.log4j:log4j-slf4j-impl to classpath



---------






* Added Slack webhook URL validation regex




* Replaced wrongly formatted dummy Slack URL with properly formatted dummy Slack URL





* Replaced more wrongly formatted dummy Slack URL with properly formatted dummy Slack URL




* Replaced even more wrongly formatted dummy Slack URL with properly formatted dummy Slack URL




* Replace path of mock Slack URL with `sample_slack_url`




* Remove slackId from domainIds




* Replace wrongly formatted dummy Slack URL with properly formatted dummy Slack URL




* Add tests for wrong Slack URLs



* Add validation tests for Slack URL



* Format



* GovSlack apps can use the slack-gov.com domain



* Add validation for gov-slack.com domain



---------
























(cherry picked from commit 87109a0)

Signed-off-by: danielkyalo599 <johbiento@gmail.com>
Signed-off-by: zhichao-aws <zhichaog@amazon.com>
Signed-off-by: Aniruddh <ansriv2005@gmail.com>
Signed-off-by: Hailong Cui <ihailong@amazon.com>
Signed-off-by: Darshit Chanpura <dchanp@amazon.com>
Signed-off-by: yuye-aws <yuyezhu@amazon.com>
Signed-off-by: gaobinlong <gbinlong@amazon.com>
Signed-off-by: Aniruddh Srivastava <aniruddh@purdue.edu>
Signed-off-by: Aniruddh <63553175+Noir01@users.noreply.github.com>
Signed-off-by: rdani <rachana_sudhindradani@intuit.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: Noir <63553175+Noir01@users.noreply.github.com>
Signed-off-by: Aniruddh Srivastava <ansriv2005@gmail.com>
Signed-off-by: Aniruddh <aniruddh@purdue.edu>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: zhichao-aws <zhichaog@amazon.com>
Co-authored-by: danielkyalo599 <johbiento@gmail.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
Co-authored-by: Darshit Chanpura <35282393+DarshitChanpura@users.noreply.github.com>
Co-authored-by: Yuye Zhu <yuyezhu@amazon.com>
Co-authored-by: gaobinlong <gbinlong@amazon.com>
Co-authored-by: Rachana Dani <36135368+rachana-dani@users.noreply.github.com>
Co-authored-by: rdani <rachana_sudhindradani@intuit.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants