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

update notifications whitelist #6258

Closed
absynth76 opened this issue Mar 11, 2020 · 1 comment
Closed

update notifications whitelist #6258

absynth76 opened this issue Mar 11, 2020 · 1 comment
Assignees
@nixocio @kdelee
Labels
component:api type:enhancement

Comments

@absynth76
Copy link

ISSUE TYPE
  • Feature Idea
SUMMARY

One of the main deployment information is the tag and/or branch version, or the scm_branch parameter is not available through the job dictionary and so notifications cannot include that common field. Could you please add it to the JOB_FIELDS_WHITELIST in notifications.yml? thank you.
@wenottingham wenottingham mentioned this issue Mar 17, 2020
Merged
@kdelee
Copy link
Member

kdelee commented Mar 17, 2020

Works for us! Made notifications w/ this var and verified that it used to throw 400, but now it works.

@kdelee kdelee closed this as completed Mar 17, 2020
TheRealHaoLiu pushed a commit to TheRealHaoLiu/awx that referenced this issue Feb 8, 2023
@Zokormazo
Merge pull request ansible#6258 from AlanCoding/bad_character_43
4d95abc 
[4.3] Workaround for events with NUL char, touch up error loop
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nixocio nixocio

@kdelee kdelee

Labels
component:api type:enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@ryanpetrello @AlanCoding @wenottingham @nixocio @kdelee @absynth76