Commit a253318
feat: added OAuth 2.0 authentication with PKCE and automatic token refresh (#66)
* Initial plan
* Add OAuth 2.0 authentication types and core functionality
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
* Add OAuth 2.0 styling, tests, and documentation
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
* Add help link and OAuth2 provider examples (Azure, AWS, Keycloak)
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
* Document OAuth2 redirect URI registration requirement
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
* feat: added handling of id tokens
* chore: reset this bit to main
* Update packages/yasgui/src/Tab.ts
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update docs/developer-guide.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update packages/yasgui/src/TabSettingsModal.ts
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Fix OAuth2 race conditions, memory leaks, and improve UX
- Add unique flowId to prevent sessionStorage collisions across multiple OAuth flows
- Add flowCompleted flag to prevent race condition between polling and messageHandler
- Fix memory leaks by removing event listeners in all error paths
- Improve alert messages with specific instructions on accessing Settings
- Consolidate cleanup logic to prevent duplicate sessionStorage operations
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
* Refactor: Extract closeModalAndRefresh helper to reduce duplication
Extract repeated pattern of closing auth modal and refreshing endpoints list into a helper function. This reduces code duplication across all authentication types (basic, bearer, apiKey, oauth2) and improves maintainability.
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
---------
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: MathiasVDA <15101339+MathiasVDA@users.noreply.github.com>
Co-authored-by: Mathias Vanden Auweele <mathias@matdata.eu>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>1 parent 4517d24 commit a253318
File tree
12 files changed
+1171
-24
lines changed- docs
- packages
- yasgui/src
- yasqe/src
- __tests__
12 files changed
+1171
-24
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1014 | 1014 | | |
1015 | 1015 | | |
1016 | 1016 | | |
| 1017 | + | |
1017 | 1018 | | |
1018 | 1019 | | |
1019 | 1020 | | |
| |||
1029 | 1030 | | |
1030 | 1031 | | |
1031 | 1032 | | |
| 1033 | + | |
| 1034 | + | |
| 1035 | + | |
| 1036 | + | |
| 1037 | + | |
1032 | 1038 | | |
1033 | 1039 | | |
1034 | 1040 | | |
| |||
1067 | 1073 | | |
1068 | 1074 | | |
1069 | 1075 | | |
1070 | | - | |
| 1076 | + | |
1071 | 1077 | | |
1072 | 1078 | | |
1073 | 1079 | | |
1074 | 1080 | | |
| 1081 | + | |
1075 | 1082 | | |
1076 | 1083 | | |
1077 | 1084 | | |
| |||
1159 | 1166 | | |
1160 | 1167 | | |
1161 | 1168 | | |
| 1169 | + | |
| 1170 | + | |
| 1171 | + | |
| 1172 | + | |
| 1173 | + | |
| 1174 | + | |
| 1175 | + | |
| 1176 | + | |
| 1177 | + | |
| 1178 | + | |
| 1179 | + | |
| 1180 | + | |
| 1181 | + | |
| 1182 | + | |
| 1183 | + | |
| 1184 | + | |
| 1185 | + | |
| 1186 | + | |
1162 | 1187 | | |
1163 | 1188 | | |
1164 | 1189 | | |
| |||
1171 | 1196 | | |
1172 | 1197 | | |
1173 | 1198 | | |
| 1199 | + | |
| 1200 | + | |
| 1201 | + | |
| 1202 | + | |
| 1203 | + | |
| 1204 | + | |
| 1205 | + | |
| 1206 | + | |
| 1207 | + | |
| 1208 | + | |
| 1209 | + | |
| 1210 | + | |
| 1211 | + | |
| 1212 | + | |
| 1213 | + | |
| 1214 | + | |
| 1215 | + | |
| 1216 | + | |
| 1217 | + | |
| 1218 | + | |
| 1219 | + | |
| 1220 | + | |
| 1221 | + | |
| 1222 | + | |
| 1223 | + | |
| 1224 | + | |
| 1225 | + | |
| 1226 | + | |
| 1227 | + | |
| 1228 | + | |
| 1229 | + | |
| 1230 | + | |
| 1231 | + | |
| 1232 | + | |
| 1233 | + | |
| 1234 | + | |
| 1235 | + | |
| 1236 | + | |
| 1237 | + | |
| 1238 | + | |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
| 1248 | + | |
| 1249 | + | |
| 1250 | + | |
| 1251 | + | |
| 1252 | + | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
| 1260 | + | |
| 1261 | + | |
| 1262 | + | |
| 1263 | + | |
| 1264 | + | |
1174 | 1265 | | |
1175 | 1266 | | |
1176 | 1267 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
709 | 709 | | |
710 | 710 | | |
711 | 711 | | |
712 | | - | |
| 712 | + | |
713 | 713 | | |
714 | 714 | | |
715 | 715 | | |
716 | | - | |
717 | | - | |
718 | | - | |
719 | | - | |
720 | | - | |
| 716 | + | |
| 717 | + | |
| 718 | + | |
| 719 | + | |
| 720 | + | |
| 721 | + | |
| 722 | + | |
| 723 | + | |
| 724 | + | |
| 725 | + | |
| 726 | + | |
| 727 | + | |
| 728 | + | |
| 729 | + | |
| 730 | + | |
| 731 | + | |
| 732 | + | |
| 733 | + | |
| 734 | + | |
| 735 | + | |
| 736 | + | |
| 737 | + | |
| 738 | + | |
| 739 | + | |
| 740 | + | |
| 741 | + | |
| 742 | + | |
| 743 | + | |
| 744 | + | |
| 745 | + | |
| 746 | + | |
| 747 | + | |
| 748 | + | |
721 | 749 | | |
722 | 750 | | |
723 | 751 | | |
724 | 752 | | |
725 | 753 | | |
726 | | - | |
| 754 | + | |
727 | 755 | | |
728 | 756 | | |
| 757 | + | |
| 758 | + | |
729 | 759 | | |
730 | 760 | | |
731 | 761 | | |
| |||
735 | 765 | | |
736 | 766 | | |
737 | 767 | | |
| 768 | + | |
| 769 | + | |
738 | 770 | | |
739 | 771 | | |
740 | 772 | | |
741 | 773 | | |
| 774 | + | |
| 775 | + | |
| 776 | + | |
| 777 | + | |
| 778 | + | |
| 779 | + | |
| 780 | + | |
| 781 | + | |
| 782 | + | |
| 783 | + | |
| 784 | + | |
| 785 | + | |
| 786 | + | |
| 787 | + | |
| 788 | + | |
| 789 | + | |
| 790 | + | |
| 791 | + | |
| 792 | + | |
| 793 | + | |
| 794 | + | |
| 795 | + | |
| 796 | + | |
| 797 | + | |
| 798 | + | |
| 799 | + | |
| 800 | + | |
| 801 | + | |
| 802 | + | |
| 803 | + | |
| 804 | + | |
| 805 | + | |
| 806 | + | |
| 807 | + | |
| 808 | + | |
| 809 | + | |
| 810 | + | |
| 811 | + | |
| 812 | + | |
| 813 | + | |
| 814 | + | |
| 815 | + | |
| 816 | + | |
| 817 | + | |
| 818 | + | |
742 | 819 | | |
743 | 820 | | |
744 | 821 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
307 | 307 | | |
308 | 308 | | |
309 | 309 | | |
| 310 | + | |
310 | 311 | | |
311 | 312 | | |
312 | 313 | | |
| |||
0 commit comments