Skip to content

Commit c0462e5

Browse files
authored
Refactor request URL construction for batch sensitivity label assignment (#5108)
1 parent 904898a commit c0462e5

File tree

1 file changed

+4
-14
lines changed

1 file changed

+4
-14
lines changed

src/Commands/Files/AddFileSensitivityLabel.cs

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ namespace PnP.PowerShell.Commands.Files
1717
[Cmdlet(VerbsCommon.Add, "PnPFileSensitivityLabel", DefaultParameterSetName = ParameterSet_SINGLE)]
1818
[RequiredApiDelegatedOrApplicationPermissions("graph/Files.ReadWrite.All")]
1919
[RequiredApiDelegatedOrApplicationPermissions("graph/Sites.ReadWrite.All")]
20-
2120
public class AddFileSensitivityLabel : PnPGraphCmdlet
2221
{
2322
private const string ParameterSet_SINGLE = "Single";
@@ -51,11 +50,12 @@ protected override void ExecuteCmdlet()
5150
IFile file = Identity.GetCoreFile(context, this);
5251
file.EnsureProperties(f => f.VroomDriveID, f => f.VroomItemID, f => f.Name);
5352

54-
var requestUrl = GetRequestUrl(file);
53+
var requestUrl = $"v1.0/drives/{file.VroomDriveID}/items/{file.VroomItemID}/assignSensitivityLabel";
5554
var payloadJson = SerializePayload();
5655

5756
if (ParameterSpecified(nameof(Batch)))
5857
{
58+
requestUrl = $"drives/{file.VroomDriveID}/items/{file.VroomItemID}/assignSensitivityLabel";
5959
QueueBatchRequest(requestUrl, payloadJson, file);
6060
}
6161
else
@@ -80,12 +80,7 @@ private void QueueBatchRequest(string requestUrl, string payloadJson, IFile file
8080
new ApiRequest(HttpMethod.Post, ApiRequestType.Graph, requestUrl, payloadJson));
8181

8282
LogDebug($"Queued file sensitivity label assignment for {file.Name}");
83-
}
84-
85-
private static string GetRequestUrl(IFile file)
86-
{
87-
return $"v1.0/drives/{file.VroomDriveID}/items/{file.VroomItemID}/assignSensitivityLabel";
88-
}
83+
}
8984

9085
private string SerializePayload()
9186
{
@@ -96,12 +91,7 @@ private string SerializePayload()
9691
justificationText = JustificationText
9792
};
9893

99-
var serializerOptions = new JsonSerializerOptions
100-
{
101-
DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull
102-
};
103-
104-
return JsonSerializer.Serialize(payload, serializerOptions);
94+
return JsonSerializer.Serialize(payload);
10595
}
10696
}
10797
}

0 commit comments

Comments
 (0)