Hi kevin,

1. you need eupdated sql to directly overwrite the sql in [1] (h2,
mysql, pg), and create 2 upgrade sql file(Includes mysql and pg)  in
[2] .
    like 2.4.2-upgrade-2.4.3-mysql.sql and 2.4.2-upgrade-2.4.3-pg.sql.

2.  Maybe  encrypt directly with sha-512  is better (method 2)

[1]: 
https://github.com/apache/incubator-shenyu/tree/master/shenyu-admin/src/main/resources/sql-script
[2]: https://github.com/apache/incubator-shenyu/tree/master/script

Kevin <[email protected]> 于2022年2月22日周二 15:38写道:
>
> Hi,XiaoYu
> Thanks for reminding me about updating sql.
> Do I need to submit the updated sql to directly overwrite the sql in [1]?
> Or provide a 2.4.2-upgrade-2.4.3-mysql.sql somewhere specified?
>
> about what you mentioned
>
> I think used SHA-512 with encrypt the password is better.
>
>
> Method 1: Do you mean like the aes encryption function to encrypt the text
> with a specified key?
>  SHA-512[1] does not support using a key to encrypt text, but we can add a
> specified key to the text to generate encrypted text.
>
> Method 2 : Maybe you meant to encrypt directly with sha-512?
>
> I'm not quite sure if your opinion is method 1 or method 2, so wanted to
> check with you.
>
> Regards
>
> [1]:
> https://github.com/apache/incubator-shenyu/tree/master/shenyu-admin/src/main/resources/sql-script
> [2]:https://en.wikipedia.org/wiki/SHA-2
>
>
>
> XiaoYu <[email protected]> 于2022年2月22日周二 14:59写道:
>
> > Hi Kevin
> >
> > I have carefully read through your improvement plan for this issue[1].
> >
> > I think used SHA-512 with encrypt the password is better.
> >
> > However, please note that the relevant SQL[2] are updated (insert into
> > dashboard_user),the default password is 123456.
> >
> > As there are sql statements involved, you will need to create the sql
> > upgrade file,like 2.4.2-upgrade-2.4.3-mysql.sql or
> > 2.4.2-upgrade-2.4.3-pg.sql in [3].
> >
> > Looking forward to your  pull request.
> >
> > Regards
> >
> >
> > [1]: https://github.com/apache/incubator-shenyu/issues/1888
> > [2]:
> > https://github.com/apache/incubator-shenyu/tree/master/shenyu-admin/src/main/resources/sql-script
> > [3]: https://github.com/apache/incubator-shenyu/tree/master/script
> >
> > Kevin <[email protected]> 于2022年2月22日周二 14:41写道:
> > >
> > > Dear community friends:
> > >
> > > I answered a long time ago issue with a solution.
> > > But I'm not sure if my solution is suitable for shenyu's needs.
> > > If you are free, please see my solution and help me solve the problem
> > > better.
> > >
> > > Issue: https://github.com/apache/incubator-shenyu/issues/1888
> > >
> > > Regards
> >

Reply via email to