Skip to content

Commit

Permalink
Bump to version 7 (#416)
Browse files Browse the repository at this point in the history
* Add missing Async prefix

* Bump version

* Add info on breaking change
  • Loading branch information
PeterOrneholm authored Jun 30, 2023
1 parent 048180d commit f0e0d20
Show file tree
Hide file tree
Showing 8 changed files with 9 additions and 7 deletions.
2 changes: 2 additions & 0 deletions BREAKINGCHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,9 @@ Some API changes (as described above) propagates to the options you can customiz

- `BankIdAllowBiometric` is removed and replaced by `BankIdRequirePinCode`. This has the opposite meaning as the old option, so be sure to update your code if you have used this.

### IBankIdLauncher.GetLaunchInfo is now async

The method `GetLaunchInfo` on `IBankIdLauncher` is now renamed into `GetLaunchInfoAsync` and made async to allow for dynamic retreival.

## Version 6.0.3

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ private Task<BankIdLaunchInfo> GetBankIdLaunchInfo(string redirectUrl, string au
{
var launchUrlRequest = new LaunchUrlRequest(redirectUrl, autoStartToken);

return _bankIdLauncher.GetLaunchInfo(launchUrlRequest);
return _bankIdLauncher.GetLaunchInfoAsync(launchUrlRequest);
}

public async Task<BankIdFlowCollectResult> Collect(string orderRef, int autoStartAttempts, BankIdFlowOptions flowOptions)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ internal class BankIdDevelopmentLauncher : IBankIdLauncher
{
private const string DevelopmentLaunchUrl = "#";

public Task<BankIdLaunchInfo> GetLaunchInfo(LaunchUrlRequest request)
public Task<BankIdLaunchInfo> GetLaunchInfoAsync(LaunchUrlRequest request)
{
// Always stay on same page, without reloading, in simulated mode
return Task.FromResult(new BankIdLaunchInfo(DevelopmentLaunchUrl, false, false));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public BankIdLauncher(IBankIdSupportedDeviceDetector bankIdSupportedDeviceDetect
_customAppCallbacks = customAppCallbacks.ToList();
}

public async Task<BankIdLaunchInfo> GetLaunchInfo(LaunchUrlRequest request)
public async Task<BankIdLaunchInfo> GetLaunchInfoAsync(LaunchUrlRequest request)
{
var detectedDevice = _bankIdSupportedDeviceDetector.Detect();
var deviceMightRequireUserInteractionToLaunch = GetDeviceMightRequireUserInteractionToLaunchBankIdApp(detectedDevice);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,5 @@ public interface IBankIdLauncher
/// </summary>
/// <param name="request">Launch info</param>
/// <returns></returns>
Task<BankIdLaunchInfo> GetLaunchInfo(LaunchUrlRequest request);
Task<BankIdLaunchInfo> GetLaunchInfoAsync(LaunchUrlRequest request);
}
2 changes: 1 addition & 1 deletion src/Directory.Build.props
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
<PackageId>$(AssemblyName)</PackageId>

<VersionPrefix>7.0.0</VersionPrefix>
<VersionSuffix>rc-1</VersionSuffix>
<!--<VersionSuffix>rc-1</VersionSuffix>-->
<AssemblyVersion>7.0.0.0</AssemblyVersion>
<FileVersion Condition="'$(BUILD_BUILDNUMBER)' == ''">$(VersionPrefix).0</FileVersion>
<FileVersion Condition="'$(BUILD_BUILDNUMBER)' != ''">$(VersionPrefix).$(BUILD_BUILDNUMBER)</FileVersion>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ namespace ActiveLogin.Authentication.BankId.AspNetCore.Test.Helpers;

internal class TestBankIdLauncher : IBankIdLauncher
{
public Task<BankIdLaunchInfo> GetLaunchInfo(LaunchUrlRequest request)
public Task<BankIdLaunchInfo> GetLaunchInfoAsync(LaunchUrlRequest request)
{
// Always redirect back without user interaction in simulated mode
return Task.FromResult(new BankIdLaunchInfo(request.RedirectUrl, false, false));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ namespace ActiveLogin.Authentication.BankId.Core.Test.Helpers;

internal class TestBankIdLauncher : IBankIdLauncher
{
public Task<BankIdLaunchInfo> GetLaunchInfo(LaunchUrlRequest request)
public Task<BankIdLaunchInfo> GetLaunchInfoAsync(LaunchUrlRequest request)
{
// Always redirect back without user interaction in simulated mode
return Task.FromResult(new BankIdLaunchInfo(request.RedirectUrl, false, false));
Expand Down

0 comments on commit f0e0d20

Please sign in to comment.