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.

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

fix: version updates and changelog #645

Merged
merged 13 commits into from
Jul 19, 2023
Merged

fix: version updates and changelog #645

merged 13 commits into from
Jul 19, 2023

Conversation

sattvikc
Copy link
Collaborator

Summary of change

(A few sentences about this PR)

Related issues

  • Link to issue1 here
  • Link to issue1 here

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)

Documentation changes

(If relevant, please create a PR in our docs repo, or create a checklist here highlighting the necessary changes)

Checklist for important updates

  • Changelog has been updated
  • coreDriverInterfaceSupported.json file has been updated (if needed)
    • Along with the associated array in lib/ts/version.ts
  • frontendDriverInterfaceSupported.json file has been updated (if needed)
  • Changes to the version if needed
    • In package.json
    • In package-lock.json
    • In lib/ts/version.ts
  • Had run npm run build-pretty
  • Had installed and ran the pre-commit hook
  • Issue this PR against the latest non released version branch.
    • To know which one it is, run find the latest released tag (git tag) in the format vX.Y.Z, and then find the latest branch (git branch --all) whose X.Y is greater than the latest released tag.
    • If no such branch exists, then create one from the latest released branch.
  • If have added a new web framework, update the add-ts-no-check.js file to include that
  • If added a new recipe / api interface, then make sure that the implementation of it uses NON arrow functions only (like someFunc: function () {..}).
  • If added a new recipe, then make sure to expose it inside the recipe folder present in the root of this repo. We also need to expose its types.

Remaining TODOs for this PR

  • Item1
  • Item2

@sattvikc sattvikc self-assigned this Jul 18, 2023
CHANGELOG.md Outdated
Comment on lines 109 to 110
{ clientId: "clientid1", clientSecret: "..." },
{ clientId: "clientid2", clientSecret: "..." },
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need clientType here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

},
};
```

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also mention change in how to get access token from the provider

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

lib/ts/version.ts Show resolved Hide resolved
- `oAuthTokens` - contains all the tokens (access_token, id_token, etc.) as returned by the provider
- `rawUserInfoFromProvider` - contains all the user profile info as returned by the provider
- Added `manuallyCreateOrUpdateUser` recipe function in thirdparty recipe instead, to be used in place of `signInUp`

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

list all functions that have changed in sigature - go recipe wise.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

CHANGELOG.md Outdated
@@ -7,10 +7,402 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [unreleased]

## [15.0.0]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
## [15.0.0]
## [15.0.0] - 2023-07-19

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

CHANGELOG.md Outdated
- `validateClaimsInJWTPayload`
- Added mandatory `tenantId` in the input for the following recipe interface functions. If any of these functions are overridden, they need to be updated accordingly:
- `createNewSession`
- `getGlobalClaimValidators`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

third party recipe change section is not properly indented

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

### Fixes

- Fixed an issue where certain Dashboard API routes would return a 404 for Hapi

### Migration
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do not make it 1, 2, 3.. make it a ####

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

CHANGELOG.md Outdated
4. Change in the implementation of custom providers

- All config is part of `ProviderInput`
- To update `authorisationRedirect.Params` dynamically, `getAuthorisationRedirectURL` must be overridden
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move it to the next point or move it from here entirely.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

CHANGELOG.md Outdated
};
```

Note: the SDK will fetch the oauth2 endpints from the providered OIDC discovery endpoint url + '/.well-known/openid-configuration'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

CHANGELOG.md Outdated
};
```

Note: the SDK will fetch the oauth2 endpints from the providered OIDC discovery endpoint url + '/.well-known/openid-configuration'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Properly format.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@rishabhpoddar rishabhpoddar merged commit d18a8ab into feat/tp-rework Jul 19, 2023
10 of 12 checks passed
@rishabhpoddar rishabhpoddar deleted the mt-release branch July 19, 2023 06:52
rishabhpoddar added a commit that referenced this pull request Jul 19, 2023
* fix: tp-rework interfaces, api impl

* fix: todos

* fix: iface change

* fix: further changes in thirdparty recipe

* fix: unused import

* fix: added requireEmail

* fix: minor fixes

* fix: interfaces fixes + ad apple impl

* fix: boxy, discord impl

* fix: fb impl

* fix: github impl

* fix: github impl

* fix: google and gw impl

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: updated provider impls

* fix: multitenancy recipe

* fix: recipe fixes

* fix: api impl fixes

* fix: impl fixes

* fix: tp-rework fixes

* fix: tp-rework fixes

* fix: tp-rework fixes

* fix: linkedin and okta impl

* fix: tp-rework fixes

* fix: fixed tests

* fix: fixed tests

* fix: updated for tenantId in core

* fix: rebuild

* fix: router to handle tenantid (#592)

* fix: router to handle tenantid

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: multitenancy recipe (#594)

* fix: impl and test

* fix: version

* fix: impl

* fix: pr comments

* fix: version update

* fix: pr comments

* fix: user context refactor (#595)

* fix: first layer impl

* fix: impl

* fix: pr comment

* fix: pr comments

* fix: test for counting makeDefaultUserContext

* fix: impl (#596)

* fix: pr comment (#597)

* fix: userroles mt impl (#600)

* fix: multitenancy emailpassword impl (#602)

* fix: recipe impl

* fix: impl

* fix: password reset link

* fix: mt revert

* fix: test

* fix: pr comment

* fix: tenantId compulsory in user roles recipe interface (#605)

* fix: multitenancy passwordless implementation (#606)

* fix: impl

* fix: test

* fix: cleanup

* fix: tenantId in link

* fix: tp and ev impl

* fix: ev impl

* fix: tp test

* fix: test cross tenant resend code

* fix: tenantId in link

* fix: pr comments

* fix: optional members in UserInfo

* fix: type exports

* fix: pr comments

* fix: session recipe impl for multitenancy (#607)

* fix: impl

* fix: test

* fix: cleanup

* fix: tenantId in link

* fix: tp and ev impl

* fix: ev impl

* fix: tp test

* fix: test cross tenant resend code

* fix: tenantId in link

* fix: pr comments

* fix: recipe impl

* fix: test

* fix: create new session

* fix: test

* fix: getTenantId

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: dashboard multitenancy updates (#623)

* fix: dashboard updates

* fix: pr comments

* fix: updating send email verification interface (#625)

* fix: updating send email verification interface

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: password reset email (#626)

* fix: passwordless

* fix: password reset in tpep

* fix: rename functions

* fix: tenantid in config functions (#627)

* fix: passwordless

* fix: password reset in tpep

* fix: rename functions

* fix: tenantId in types

* fix: tenantId in types

* fix: impl

* fix: impl

* fix: pr comments

* fix: minor

* Update lib/ts/recipe/emailpassword/utils.ts

* Update lib/ts/recipe/emailverification/utils.ts

* Update lib/ts/recipe/passwordless/utils.ts

* Update lib/ts/recipe/thirdpartyemailpassword/utils.ts

* Update lib/ts/recipe/thirdpartypasswordless/utils.ts

---------

Co-authored-by: Rishabh Poddar <rishabh.poddar@gmail.com>

* fix: missed param (#629)

* fix: return updated access token after verification

* test: fix clientId casing in tests

* Update routing logic for Hapi and dashboard recipe to account for multitenancy (#631)

* fix: test fixes (#630)

* fix: test fixes

* fix: claims build

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: cleanup

* fix: tests

* fix: tests

* fix: pr comments

* fix: remove error handlers in multitenancy recipe (#632)

* fix: test fixes

* fix: claims build

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: cleanup

* fix: tests

* fix: tests

* fix: pr comments

* fix: remove error handlers in multitenancy

* fix: pr comments

* fix: pr comments

* fix: tenantid compulsory (#633)

* fix: tenantid compulsory

* fix: type fix

* fix: type fix

* fix: tenant id in get global claim validators (#634)

* fix: tenant id in global claim validators

* fix: pr comments

* refactor: Remove core config from dashboard tenants list response (#635)

* Remove core config from dashboard tenants list response

* Update tenant list API logic

* fix: Fix tenant id being passed in the wrong order for password update functions (#636)

* Remove core config from dashboard tenants list response

* Fix tenant id being passed in the wrong order for password update functions

* Add user PUT api to list

* Update tenant list API logic

* Update CHANGELOG

* fix: all tests fixes (#637)

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: test

* fix: tests

* fix: tests

* fix: test (#638)

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: tests

* fix: test

* fix: tests

* fix: tests

* fix: tests

* fix: handle tenant not found error (#639)

* fix: handle tenant not found error

* fix: pr comments

* fix: pr comments

* docs: convert next example to use app dir (#641)

* fix: bitbucket gitlab impl and other fixes (#642)

* fix: github

* fix: user info

* fix: discord and linkedin

* fix: gitlab impl

* fix: bitbucket impl

* fix: tests fixes (#643)

* fix: tests

* fix: tests

* fix: framework tests with tenantid in path

* fix: tests

* fix: google workspaces

* fix: version updates and changelog (#645)

* fix: tests

* fix: tests

* fix: framework tests with tenantid in path

* fix: tests

* fix: google workspaces

* fix: version updates

* chore: changelog

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: pr comments

* fix: auth react server (#647)

* fix: auth-react-server

* fix: auth-react-server

* fix: example test (#648)

* removes unnecessary import

---------

Co-authored-by: Rishabh Poddar <rishabh.poddar@gmail.com>
Co-authored-by: Mihaly Lengyel <mihaly@lengyel.tech>
Co-authored-by: Nemi Shah <nemishah1212@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants