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

Jump to bottom

Domain Sockets and Named Pipes #151

Merged
merged 12 commits into from
Jan 22, 2024
Merged

Domain Sockets and Named Pipes #151

merged 12 commits into from
Jan 22, 2024

Conversation

pawelvds
Copy link
Contributor

@pawelvds pawelvds commented Jan 3, 2024

No description provided.

@pawelvds pawelvds marked this pull request as ready for review January 3, 2024 14:20
@pawelvds pawelvds requested a review from a team as a code owner January 3, 2024 14:20
volllly
volllly reviewed Jan 3, 2024
View reviewed changes
src/fiskaltrust.Launcher/Commands/HostCommand.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/HostCommand.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/HostCommand.cs Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/HostCommand.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/HostCommand.cs Outdated Show resolved Hide resolved
@volllly volllly linked an issue Jan 4, 2024 that may be closed by this pull request
Use Namedpipes and UnixDomainSockets for ProcessHostService communication #112
Closed
@pawelvds pawelvds marked this pull request as draft January 4, 2024 13:51
@pawelvds pawelvds marked this pull request as ready for review January 8, 2024 23:15
@pawelvds
Modified LauncherConfiguration to include LauncherServiceUri;
77cb9d6 
Implemented ConfigureKestrel method to handle Unix Domain Sockets and Named Pipes;
@volllly volllly marked this pull request as draft January 9, 2024 14:35
@pawelvds pawelvds marked this pull request as ready for review January 9, 2024 20:21
volllly
volllly requested changes Jan 11, 2024
View reviewed changes
src/fiskaltrust.Launcher.Common/Configuration/Configuration.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher.Common/Configuration/Configuration.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher.Common/Configuration/Configuration.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher.Common/Configuration/Configuration.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/HostCommand.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/ProcessHost/ProcessHostMonarcStartup.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/ProcessHost/ProcessHostMonarcStartup.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/ProcessHost/ProcessHostMonarcStartup.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/RunCommand.cs Outdated Show resolved Hide resolved
src/fiskaltrust.Launcher/Commands/HostCommand.cs Outdated Show resolved Hide resolved
@volllly volllly marked this pull request as draft January 11, 2024 09:26
@pawelvds pawelvds marked this pull request as ready for review January 12, 2024 12:48
@pawelvds pawelvds requested a review from volllly January 12, 2024 12:49
@volllly volllly enabled auto-merge (squash) January 22, 2024 14:01
@volllly volllly merged commit 85721d9 into main Jan 22, 2024
2 of 5 checks passed
@volllly volllly deleted the Use-pipes-for-processhostservice-communication branch January 22, 2024 14:01
@github-actions github-actions bot locked and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@volllly volllly volllly approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Use Namedpipes and UnixDomainSockets for ProcessHostService communication
2 participants
@pawelvds @volllly