fix(tags): fix ownership on tag create (#2253)

datahub-project · Mar 18, 2021 · 8c8de74 · 8c8de74
1 parent d8268da
commit 8c8de74
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...ub-graphql-core/src/main/java/com/linkedin/datahub/graphql/types/dataset/DatasetType.java b/...ub-graphql-core/src/main/java/com/linkedin/datahub/graphql/types/dataset/DatasetType.java
@@ -151,7 +151,7 @@ public List<BrowsePath> browsePaths(@Nonnull String urn, @Nonnull final QueryCon
     @Override
     public Dataset update(@Nonnull DatasetUpdateInput input, @Nonnull QueryContext context) throws Exception {
         // TODO: Verify that updater is owner.
-        final CorpuserUrn actor = new CorpuserUrn(context.getActor());
+        final CorpuserUrn actor = CorpuserUrn.createFromString(context.getActor());
         final com.linkedin.dataset.Dataset partialDataset = DatasetUpdateInputMapper.map(input);
 
         // Create Audit Stamp

diff --git a/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/types/tag/TagType.java b/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/types/tag/TagType.java
@@ -115,7 +115,7 @@ public AutoCompleteResults autoComplete(@Nonnull String query,
     @Override
     public Tag update(@Nonnull TagUpdate input, @Nonnull QueryContext context) throws Exception {
         // TODO: Verify that updater is owner.
-        final CorpuserUrn actor = new CorpuserUrn(context.getActor());
+        final CorpuserUrn actor = CorpuserUrn.createFromString(context.getActor());
         final com.linkedin.tag.Tag partialTag = TagUpdateMapper.map(input);
 
         // Create Audit Stamp