From b97d6d9014adaf718b4609d993e6df9932b1ea2a Mon Sep 17 00:00:00 2001 From: 07Himank <112613760+07Himank@users.noreply.github.com> Date: Wed, 5 Apr 2023 13:14:41 +0530 Subject: [PATCH] solved url isse when space is involed in fqn (#10929) Co-authored-by: Himank Mehta Co-authored-by: Mohit Yadav <105265192+mohityadav766@users.noreply.github.com> --- .../org/openmetadata/service/util/ChangeEventParser.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/openmetadata-service/src/main/java/org/openmetadata/service/util/ChangeEventParser.java b/openmetadata-service/src/main/java/org/openmetadata/service/util/ChangeEventParser.java index e7c63151f94..b39b2341d39 100644 --- a/openmetadata-service/src/main/java/org/openmetadata/service/util/ChangeEventParser.java +++ b/openmetadata-service/src/main/java/org/openmetadata/service/util/ChangeEventParser.java @@ -200,7 +200,8 @@ public final class ChangeEventParser { } if (publishTo == PUBLISH_TO.SLACK || publishTo == PUBLISH_TO.GCHAT) { return String.format( - "<%s/%s/%s|%s>", ChangeEventConfig.getInstance().getOmUri(), entityType, fqn.trim(), fqn.trim()); + "<%s/%s/%s|%s>", + ChangeEventConfig.getInstance().getOmUri(), entityType, fqn.trim().replaceAll(" ", "%20"), fqn.trim()); } else if (publishTo == PUBLISH_TO.TEAMS) { return String.format( "[%s](/%s/%s)", fqn.trim(), ChangeEventConfig.getInstance().getOmUri(), entityType, fqn.trim()); @@ -463,7 +464,8 @@ public final class ChangeEventParser { private static String getQueryUsageUrl(PUBLISH_TO publishTo, String fqn, String entityType) { if (publishTo == PUBLISH_TO.SLACK || publishTo == PUBLISH_TO.GCHAT) { return String.format( - "<%s/%s/%s|%s>", ChangeEventConfig.getInstance().getOmUri(), entityType, fqn.trim(), fqn.trim()); + "<%s/%s/%s|%s>", + ChangeEventConfig.getInstance().getOmUri(), entityType, fqn.trim().replaceAll(" ", "%20"), fqn.trim()); } else if (publishTo == PUBLISH_TO.TEAMS) { return String.format("[%s](/%s/%s)", fqn, ChangeEventConfig.getInstance().getOmUri(), entityType, fqn.trim()); } else if (publishTo == PUBLISH_TO.EMAIL) {