Skip to content

Commit 4223eec

Browse files
committed
Remove details from WriteSessionFailure
Which is currently unused. It must have been in use previously because the client is expecting specific reasons. Since we need to redo this, let's remove the excess.
1 parent b1e468f commit 4223eec

File tree

1 file changed

+2
-9
lines changed

1 file changed

+2
-9
lines changed

src/PowerShellEditorServices.Hosting/Configuration/SessionFileWriter.cs

+2-9
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,7 @@ public interface ISessionFileWriter
2020
/// Write a session file describing a failed startup.
2121
/// </summary>
2222
/// <param name="reason">The reason for the startup failure.</param>
23-
/// <param name="details">Any details to accompany the reason.</param>
24-
void WriteSessionFailure(string reason, object details);
23+
void WriteSessionFailure(string reason);
2524

2625
/// <summary>
2726
/// Write a session file describing a successful startup.
@@ -58,8 +57,7 @@ public SessionFileWriter(HostLogger logger, string sessionFilePath)
5857
/// Write a startup failure to the session file.
5958
/// </summary>
6059
/// <param name="reason">The reason for the startup failure.</param>
61-
/// <param name="details">Any extra details, which will be serialized as JSON.</param>
62-
public void WriteSessionFailure(string reason, object details)
60+
public void WriteSessionFailure(string reason)
6361
{
6462
_logger.Log(PsesLogLevel.Diagnostic, "Writing session failure");
6563

@@ -69,11 +67,6 @@ public void WriteSessionFailure(string reason, object details)
6967
{ "reason", reason },
7068
};
7169

72-
if (details != null)
73-
{
74-
sessionObject["details"] = details;
75-
}
76-
7770
WriteSessionObject(sessionObject);
7871
}
7972

0 commit comments

Comments
 (0)