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

Fix missing dependency on Content Pack creation for Searches with Parameters #21563

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
+5 −0
Open

Fix missing dependency on Content Pack creation for Searches with Parameters #21563

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
16d8b5d
fixes missing query parameters in content packs
janheise Jan 14, 2025
aeb7996
add Nullable
janheise Jan 15, 2025
b4b4026
add Nullable
janheise Jan 15, 2025
0aa44fb
adding changelog
janheise Jan 20, 2025
c633b46
adding dependency to lookup table
janheise Jan 30, 2025
d0fda64
adding dependency to lookup table, removing obsolete imports
janheise Jan 30, 2025
4eaa1de
adding null check
janheise Jan 30, 2025
b01dd30
changelog
janheise Jan 31, 2025
d177769
Merge remote-tracking branch 'origin/master' into fix_missing_query_p…
janheise Feb 4, 2025
b87ebbd
Merge remote-tracking branch 'origin/master' into fix_missing_query_p…
janheise Feb 6, 2025
1f8911d
adding the dependency to the lookup adapter for CP creation of a Search
janheise Feb 7, 2025
04535c9
add correct changelog
janheise Feb 7, 2025
e5b9ebd
Merge remote-tracking branch 'origin/master' into fix_cp_search_missi…
janheise Feb 14, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions changelog/unreleased/pr-21563.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
type = "fixed"
message = "Fix missing dependency for Data Adapter when creating Content Pack for Search with Parameters."

pulls = ["21563"]
1 change: 1 addition & 0 deletions graylog2-server/src/main/java/org/graylog/plugins/views/search/Search.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -332,5 +332,6 @@ public SearchEntity toContentPackEntity(EntityDescriptorIds entityDescriptorIds)
@Override
public void resolveNativeEntity(EntityDescriptor entityDescriptor, MutableGraph<EntityDescriptor> mutableGraph) {
queries().forEach(query -> query.resolveNativeEntity(entityDescriptor, mutableGraph));
parameters().forEach(parameter -> parameter.resolveNativeEntity(entityDescriptor, mutableGraph));
}
}
Loading