Skip to content

Commit 2905b7d

Browse files
committed
chore: relax sendnotification options
1 parent 126f976 commit 2905b7d

7 files changed

Lines changed: 29 additions & 24 deletions

File tree

packages/client/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ console.log(client.getAccessTokenEndpoint()); // https://auth.research.identipro
6969
```
7070

7171
Using https scheme
72+
7273
```typescript
7374
import { OpenID4VCIClient } from '@sphereon/oid4vci-client';
7475

packages/client/lib/OpenID4VCIClient.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ export class OpenID4VCIClient {
473473
}
474474

475475
public async sendNotification(
476-
credentialRequestOpts: CredentialRequestOpts,
476+
credentialRequestOpts: Partial<CredentialRequestOpts>,
477477
request: NotificationRequest,
478478
accessToken?: string,
479479
): Promise<NotificationResult> {

packages/client/lib/__tests__/IssuanceInitiation.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ describe('Issuance Initiation', () => {
6868
expect(client.scheme).toEqual('https');
6969
expect(client.credential_offer.credential_issuer).toEqual('https://launchpad.vii.electron.mattrlabs.io');
7070
expect(client.preAuthorizedCode).toEqual('UPZohaodPlLBnGsqB02n2tIupCIg8nKRRUEUHWA665X');
71-
})
71+
});
7272

7373
it('Should take an http url as input and return a Credential Offer', async () => {
7474
const client = await CredentialOfferClient.fromURI(
@@ -79,5 +79,5 @@ describe('Issuance Initiation', () => {
7979
expect(client.scheme).toEqual('http');
8080
expect(client.credential_offer.credential_issuer).toEqual('http://launchpad.vii.electron.mattrlabs.io');
8181
expect(client.preAuthorizedCode).toEqual('UPZohaodPlLBnGsqB02n2tIupCIg8nKRRUEUHWA665X');
82-
})
82+
});
8383
});
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
11
export * from './AuthorizationUtil';
2+
export * from './notifications';

packages/client/lib/functions/notifications.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { CredentialRequestOpts } from '../CredentialRequestClient';
44
import { LOG } from '../types';
55

66
export async function sendNotification(
7-
credentialRequestOpts: CredentialRequestOpts,
7+
credentialRequestOpts: Partial<CredentialRequestOpts>,
88
request: NotificationRequest,
99
accessToken?: string,
1010
): Promise<NotificationResult> {

packages/common/lib/functions/Encoding.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,4 @@
1-
import {
2-
BAD_PARAMS,
3-
DecodeURIAsJsonOpts,
4-
EncodeJsonAsURIOpts,
5-
JsonURIMode,
6-
OpenId4VCIVersion,
7-
SearchValue
8-
} from '../types'
1+
import { BAD_PARAMS, DecodeURIAsJsonOpts, EncodeJsonAsURIOpts, JsonURIMode, OpenId4VCIVersion, SearchValue } from '../types';
92

103
/**
114
* @type {(json: {[s:string]: never} | ArrayLike<never> | string | object, opts?: EncodeJsonAsURIOpts)} encodes a Json object into a URI

packages/issuer/lib/__tests__/CredentialOfferUtils.spec.ts

Lines changed: 22 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,17 @@ describe('CredentialOfferUtils should', () => {
4343
},
4444
} as CredentialOfferPayloadV1_0_11
4545

46-
expect(createCredentialOfferURI({
47-
credential_issuer: credentialOffer.credential_issuer,
48-
credential_endpoint: 'test_issuer',
49-
issuer: 'test_issuer',
50-
credentials_supported: []
51-
}, { credentialOffer, state: 'eyJhbGciOiJSU0Et...FYUaBy', scheme: 'https' })).toEqual(
46+
expect(
47+
createCredentialOfferURI(
48+
{
49+
credential_issuer: credentialOffer.credential_issuer,
50+
credential_endpoint: 'test_issuer',
51+
issuer: 'test_issuer',
52+
credentials_supported: [],
53+
},
54+
{ credentialOffer, state: 'eyJhbGciOiJSU0Et...FYUaBy', scheme: 'https' },
55+
),
56+
).toEqual(
5257
`${credentialOffer.credential_issuer}?credential_offer=%7B%22credential_issuer%22%3A%22https%3A%2F%2Fcredential-issuer.example.com%22%2C%22credentials%22%3A%5B%7B%22format%22%3A%22jwt_vc_json%22%2C%22types%22%3A%5B%22VerifiableCredential%22%2C%22UniversityDegreeCredential%22%5D%7D%5D%2C%22grants%22%3A%7B%22authorization_code%22%3A%7B%22issuer_state%22%3A%22eyJhbGciOiJSU0Et...FYUaBy%22%7D%7D%7D`,
5358
)
5459
})
@@ -71,12 +76,17 @@ describe('CredentialOfferUtils should', () => {
7176
},
7277
} as CredentialOfferPayloadV1_0_11
7378

74-
expect(createCredentialOfferURI({
75-
credential_issuer: credentialOffer.credential_issuer,
76-
credential_endpoint: 'test_issuer',
77-
issuer: 'test_issuer',
78-
credentials_supported: []
79-
}, { credentialOffer, state: 'eyJhbGciOiJSU0Et...FYUaBy', scheme: 'http' })).toEqual(
79+
expect(
80+
createCredentialOfferURI(
81+
{
82+
credential_issuer: credentialOffer.credential_issuer,
83+
credential_endpoint: 'test_issuer',
84+
issuer: 'test_issuer',
85+
credentials_supported: [],
86+
},
87+
{ credentialOffer, state: 'eyJhbGciOiJSU0Et...FYUaBy', scheme: 'http' },
88+
),
89+
).toEqual(
8090
`${credentialOffer.credential_issuer}?credential_offer=%7B%22credential_issuer%22%3A%22http%3A%2F%2Fcredential-issuer.example.com%22%2C%22credentials%22%3A%5B%7B%22format%22%3A%22jwt_vc_json%22%2C%22types%22%3A%5B%22VerifiableCredential%22%2C%22UniversityDegreeCredential%22%5D%7D%5D%2C%22grants%22%3A%7B%22authorization_code%22%3A%7B%22issuer_state%22%3A%22eyJhbGciOiJSU0Et...FYUaBy%22%7D%7D%7D`,
8191
)
8292
})

0 commit comments

Comments
 (0)