Fix jwks and formatOriginalToken issues

src/pages/IstioConfigNew/RequestAuthenticationForm.tsx

@@ -119,6 +119,8 @@ class RequestAuthenticationForm extends React.Component<Props, RequestAuthentica
   };
 
   onAddJwtRule = (jwtRule: JWTRule) => {
+    console.log('TODELETE onAddJwtRule');
+    console.log(jwtRule);
     this.setState(
       (prevState) => {
         prevState.jwtRules.push(jwtRule);

src/pages/IstioConfigNew/RequestAuthorizationForm/JwtRuleBuilder.tsx

@@ -6,6 +6,7 @@ import { PlusCircleIcon } from '@patternfly/react-icons';
 import { TextInputBase as TextInput } from '@patternfly/react-core/dist/js/components/TextInput/TextInput';
 import { style } from 'typestyle';
 import { PfColors } from '../../../components/Pf/PfColors';
+import { isValidUrl } from '../../../utils/IstioConfigUtils';
 
 type Props = {
   onAddJwtRule: (rule: JWTRule) => void;
@@ -83,7 +84,7 @@ export const formatJwtField = (jwtField: string, jwtRule: JWTRule): string => {
     case 'outputPayloadToHeader':
       return jwtRule.outputPayloadToHeader ? jwtRule.outputPayloadToHeader : '';
     case 'forwardOriginalToken':
-      return jwtRule.forwardOriginalToken ? '' + jwtRule.forwardOriginalToken : '';
+      return jwtRule.forwardOriginalToken ? '' + jwtRule.forwardOriginalToken : 'false';
     default:
   }
   return '';
@@ -216,6 +217,9 @@ class JwtRuleBuilder extends React.Component<Props, State> {
     if (isEmptyValue) {
       return [false, 'Value cannot be empty'];
     }
+    if (this.state.newJwtField === 'jwksUri' && !isValidUrl(this.state.newValues)) {
+      return [false, 'jwsUri is not a valid Uri'];
+    }
     return [true, ''];
   };
 

src/pages/IstioConfigNew/RequestAuthorizationForm/JwtRuleList.tsx

@@ -46,6 +46,11 @@ class JwtRuleList extends React.Component<Props> {
                 <b>audiences</b>: [{formatJwtField('audiences', jwtRule)}]
               </div>
             ) : undefined}
+            {jwtRule.jwks ? (
+              <div>
+                <b>jwks</b>: [{formatJwtField('jwks', jwtRule)}]
+              </div>
+            ) : undefined}
             {jwtRule.jwksUri ? (
               <div>
                 <b>jwksUri</b>: [{formatJwtField('jwksUri', jwtRule)}]
@@ -66,7 +71,7 @@ class JwtRuleList extends React.Component<Props> {
                 <b>outputPayloadToHeader</b>: [{formatJwtField('outputPayloadToHeader', jwtRule)}]
               </div>
             ) : undefined}
-            {jwtRule.forwardOriginalToken ? (
+            {jwtRule.forwardOriginalToken !== undefined ? (
               <div>
                 <b>forwardOriginalToken</b>: [{formatJwtField('forwardOriginalToken', jwtRule)}]
               </div>

src/utils/IstioConfigUtils.ts

@@ -126,3 +126,12 @@ export const isServerHostValid = (serverHost: string, nsMandatory: boolean): boo
 export const isValidIp = (ip: string): boolean => {
   return ipRegexp.test(ip);
 };
+
+export const isValidUrl = (url: string): boolean => {
+  try {
+    new URL(url);
+  } catch (_) {
+    return false;
+  }
+  return true;
+};

src/utils/__tests__/IstioConfigUtils.test.ts

@@ -1,4 +1,4 @@
-import { isServerHostValid, mergeJsonPatch } from '../IstioConfigUtils';
+import { isServerHostValid, isValidUrl, mergeJsonPatch } from '../IstioConfigUtils';
 
 describe('Validate JSON Patchs', () => {
   const gateway: object = {
@@ -83,3 +83,15 @@ describe('Validate Gateway/Sidecar Server Host ', () => {
     expect(isServerHostValid('bookinf*/reviews', true)).toBeFalsy();
   });
 });
+
+describe('Validate bad urls', () => {
+  it('Good urls', () => {
+    expect(isValidUrl('http://www.googleapis.com/oauth2/v1/certs')).toBeTruthy();
+    expect(isValidUrl('https://www.googleapis.com/oauth2/v1/certs')).toBeTruthy();
+  });
+
+  it('Bad urls', () => {
+    expect(isValidUrl('ramdom')).toBeFalsy();
+    expect(isValidUrl('123test')).toBeFalsy();
+  });
+});