From e90daa7ec766b5aa2f8fa1ed6e8b2f9d5a49f6a3 Mon Sep 17 00:00:00 2001
From: wmdmomo <wangpersist@163.com>
Date: Wed, 21 Jul 2021 22:54:47 +0800
Subject: [PATCH] feat(plugin): fix UI limit-conn

---
 web/src/components/Plugin/PluginDetail.tsx  |  3 ++-
 web/src/components/Plugin/UI/limit-conn.tsx | 16 +++++++++-------
 web/src/components/Plugin/UI/plugin.tsx     |  5 +++--
 3 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/web/src/components/Plugin/PluginDetail.tsx b/web/src/components/Plugin/PluginDetail.tsx
index 41480125e5..882a910c7a 100644
--- a/web/src/components/Plugin/PluginDetail.tsx
+++ b/web/src/components/Plugin/PluginDetail.tsx
@@ -107,6 +107,7 @@ const PluginDetail: React.FC<Props> = ({
   const [UIForm] = Form.useForm();
   const data = initialData[name] || {};
   const pluginType = pluginList.find((item) => item.name === name)?.originType;
+  const pluginSchema = pluginList.find((item) => item.name === name)?.schema;
   const [content, setContent] = useState<string>(JSON.stringify(data, null, 2));
   const [monacoMode, setMonacoMode] = useState<PluginComponent.MonacoLanguage>(monacoModeList.JSON);
   const modeOptions: { label: string; value: string }[] = [
@@ -411,7 +412,7 @@ const PluginDetail: React.FC<Props> = ({
           </Button>
         ]}
       />
-      {Boolean(monacoMode === monacoModeList.UIForm) && <PluginForm name={name} form={UIForm} renderForm={!(pluginType === 'auth' && schemaType !== 'consumer')} />}
+      {Boolean(monacoMode === monacoModeList.UIForm) && <PluginForm name={name} schema={pluginSchema} form={UIForm} renderForm={!(pluginType === 'auth' && schemaType !== 'consumer')} />}
       <div style={{ display: monacoMode === monacoModeList.UIForm ? 'none' : 'unset' }}>
         <Editor
           value={content}
diff --git a/web/src/components/Plugin/UI/limit-conn.tsx b/web/src/components/Plugin/UI/limit-conn.tsx
index ec535fc889..3758427895 100644
--- a/web/src/components/Plugin/UI/limit-conn.tsx
+++ b/web/src/components/Plugin/UI/limit-conn.tsx
@@ -21,6 +21,7 @@ import { useIntl } from 'umi';
 
 type Props = {
   form: FormInstance;
+  schema: Record<string, any> | undefined;
   ref?: any;
 };
 
@@ -33,8 +34,9 @@ const FORM_ITEM_LAYOUT = {
   },
 };
 
-const LimitConn: React.FC<Props> = ({ form }) => {
+const LimitConn: React.FC<Props> = ({ form, schema }) => {
   const { formatMessage } = useIntl();
+  const propertires = schema?.properties
   return (
     <Form
       form={form}
@@ -46,7 +48,7 @@ const LimitConn: React.FC<Props> = ({ form }) => {
         name="conn"
         tooltip={formatMessage({ id: 'component.pluginForm.limit-conn.conn.tooltip' })}
       >
-        <InputNumber min={1} required />
+        <InputNumber min={propertires.conn.exclusiveMinimum} required />
       </Form.Item>
       <Form.Item
         label="burst"
@@ -54,7 +56,7 @@ const LimitConn: React.FC<Props> = ({ form }) => {
         name="burst"
         tooltip={formatMessage({ id: 'component.pluginForm.limit-conn.burst.tooltip' })}
       >
-        <InputNumber min={0} required />
+        <InputNumber min={propertires.burst.minimum} required />
       </Form.Item>
       <Form.Item
         label="default_conn_delay"
@@ -62,7 +64,7 @@ const LimitConn: React.FC<Props> = ({ form }) => {
         name="default_conn_delay"
         tooltip={formatMessage({ id: 'component.pluginForm.limit-conn.default_conn_delay.tooltip' })}
       >
-        <InputNumber step={0.001} min={0.001} required />
+        <InputNumber step={0.001} min={propertires.default_conn_delay.exclusiveMinimum} required />
       </Form.Item>
 
       <Form.Item
@@ -72,7 +74,7 @@ const LimitConn: React.FC<Props> = ({ form }) => {
         tooltip={formatMessage({ id: 'component.pluginForm.limit-conn.key.tooltip' })}
       >
         <Select>
-          {["remote_addr", "server_addr", "http_x_real_ip", "http_x_forwarded_for", "consumer_name"].map(item => {
+          {propertires.key.enum.map((item: string) => {
             return <Select.Option value={item} key={item}>{item}</Select.Option>
           })}
         </Select>
@@ -81,10 +83,10 @@ const LimitConn: React.FC<Props> = ({ form }) => {
       <Form.Item
         label="rejected_code"
         name="rejected_code"
-        initialValue={503}
+        initialValue={propertires.rejected_code.default}
         tooltip={formatMessage({ id: 'component.pluginForm.limit-conn.rejected_code.tooltip' })}
       >
-        <InputNumber min={200} max={599} required />
+        <InputNumber min={propertires.rejected_code.minimum} max={propertires.rejected_code.maximum} required />
       </Form.Item>
     </Form>
   );
diff --git a/web/src/components/Plugin/UI/plugin.tsx b/web/src/components/Plugin/UI/plugin.tsx
index 0d1a2b67b2..a28c42b7f2 100644
--- a/web/src/components/Plugin/UI/plugin.tsx
+++ b/web/src/components/Plugin/UI/plugin.tsx
@@ -30,13 +30,14 @@ import Cors from './cors';
 
 type Props = {
   name: string,
+  schema: Record<string, any> | undefined,
   form: FormInstance,
   renderForm: boolean
 }
 
 export const PLUGIN_UI_LIST = ['api-breaker', 'basic-auth', 'cors', 'limit-req', 'limit-conn', 'proxy-mirror', 'referer-restriction', 'limit-count'];
 
-export const PluginForm: React.FC<Props> = ({ name, renderForm, form }) => {
+export const PluginForm: React.FC<Props> = ({ name, schema, renderForm, form }) => {
 
   const { formatMessage } = useIntl();
 
@@ -56,7 +57,7 @@ export const PluginForm: React.FC<Props> = ({ name, renderForm, form }) => {
     case 'proxy-mirror':
       return <ProxyMirror form={form} />
     case 'limit-conn':
-      return <LimitConn form={form} />
+      return <LimitConn form={form} schema={schema}/>
     case 'referer-restriction':
       return <RefererRestriction form={form} />
     default: