-
Notifications
You must be signed in to change notification settings - Fork 38
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.
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
Decode Base64 usernames/passwords as UTF-8 strings #269
Merged
Merged
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
29db3b9
* fixed user credentials encoding
RagingKore 16dd1b9
* tests: removed linked files and added new project for common code
RagingKore 9dcb10f
* removed solution file leftovers of another project
RagingKore c340a42
* upgraded ConfigureAwait analyzer
RagingKore fa00978
* nicer fixture with all clients
RagingKore 33512f3
* implicit usings on tests and file scoped namespaces
RagingKore e39d52d
* no more IClassFixture
RagingKore 67e9132
* more refactored tests.
RagingKore 41f7f7e
* fixed some tests
RagingKore 816361f
* removed dotnet 5 from ci scripts
RagingKore 2fcc7f5
* added System.Text.RegularExpressions explicitly in tests to avoid e…
RagingKore 8920870
* removed generators reference in test props
RagingKore 5189685
* quickly added some assembly info files to disable parallel test...
RagingKore e8e3182
* attempt to improve docker isolation in tests
RagingKore 35bcee4
* isolated the shutdown node test
RagingKore 8f33893
* optimized warmup retry delays
RagingKore 305c857
* removed zombie packages
RagingKore 0604bba
* another attempt
RagingKore 9b27aaf
* increased health check timeout
RagingKore 4b248e7
* go go go
RagingKore 8753089
* disabled all parallel tests and adjusted the tooling.
RagingKore 477c03a
* small stuff
RagingKore 418e996
* generate simple passwords for lts compatibility (this might be it)
RagingKore 9ef5f57
* re-activating warmup logic
RagingKore ba3de54
* is it the warmup now?
RagingKore 3228da0
* activated parallel test for operations and users
RagingKore 17bcbf4
* disabled parallel tests. at this point CI hates them.
RagingKore 609f706
* disable fail fast for tests
RagingKore 89dd71a
* can't use log message for checking container health
RagingKore 78db186
reverted the removal of dotnet 5
RagingKore 766198a
* fixed .net 5 missing references and exceptions
RagingKore 42077a0
* added back net5.0 target to samples
RagingKore 82ad41b
* fixed samples build
RagingKore 519ec26
* removed AuthenticationHeaderValue ctor from user credentials (and t…
RagingKore File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,15 +1,15 @@ | ||
name: "Dispatch" | ||
|
||
on: | ||
workflow_dispatch: | ||
inputs: | ||
version: | ||
description: "Docker tag version" | ||
required: true | ||
type: string | ||
workflow_dispatch: | ||
inputs: | ||
version: | ||
description: "Docker tag version" | ||
required: true | ||
type: string | ||
|
||
jobs: | ||
test: | ||
uses: ./.github/workflows/base.yml | ||
with: | ||
docker-tag: ${{ inputs.version }} | ||
test: | ||
uses: ./.github/workflows/base.yml | ||
with: | ||
docker-tag: ${{ inputs.version }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,20 @@ | ||
<Project> | ||
<PropertyGroup> | ||
<TargetFrameworks>net5.0;net6.0;net7.0</TargetFrameworks> | ||
<Platform>x64</Platform> | ||
<GenerateAssemblyInfo>true</GenerateAssemblyInfo> | ||
<Nullable>enable</Nullable> | ||
<NullableContextOptions>enable</NullableContextOptions> | ||
<TreatWarningsAsErrors>true</TreatWarningsAsErrors> | ||
<LangVersion>11</LangVersion> | ||
<Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration> | ||
<DebugType Condition=" '$(Configuration)' == 'Debug' ">full</DebugType> | ||
<DebugType Condition=" '$(Configuration)' == 'Release' ">pdbonly</DebugType> | ||
<OutputPath>$(MSBuildThisFileDirectory)\bin\$(Configuration)\$(MSBuildProjectName)\</OutputPath> | ||
<RootNamespace>EventStore.Client</RootNamespace> | ||
<UseLinkBase>true</UseLinkBase> | ||
<GrpcPackageVersion>2.49.0</GrpcPackageVersion> | ||
<GrpcToolsPackageVersion>2.50.0</GrpcToolsPackageVersion> | ||
</PropertyGroup> | ||
<PropertyGroup> | ||
<TargetFrameworks>net5.0;net6.0;net7.0;</TargetFrameworks> | ||
<Platform>x64</Platform> | ||
<GenerateAssemblyInfo>true</GenerateAssemblyInfo> | ||
<Nullable>enable</Nullable> | ||
<NullableContextOptions>enable</NullableContextOptions> | ||
<ImplicitUsings>true</ImplicitUsings> | ||
<TreatWarningsAsErrors>true</TreatWarningsAsErrors> | ||
<LangVersion>preview</LangVersion> | ||
|
||
<Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration> | ||
<DebugType Condition=" '$(Configuration)' == 'Debug' ">full</DebugType> | ||
<DebugType Condition=" '$(Configuration)' == 'Release' ">pdbonly</DebugType> | ||
|
||
<UseLinkBase>true</UseLinkBase> | ||
<GrpcPackageVersion>2.49.0</GrpcPackageVersion> | ||
<GrpcToolsPackageVersion>2.50.0</GrpcToolsPackageVersion> | ||
</PropertyGroup> | ||
</Project> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
<Project> | ||
<PropertyGroup> | ||
<TreatWarningsAsErrors>true</TreatWarningsAsErrors> | ||
</PropertyGroup> | ||
<PropertyGroup> | ||
<TreatWarningsAsErrors>true</TreatWarningsAsErrors> | ||
</PropertyGroup> | ||
</Project> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did tabs get changed to spaces here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will fix on another PR of "cosmetic" nature. And yes I'm struggling with tabs a lot XD