-
Notifications
You must be signed in to change notification settings - Fork 2
Tring react #15
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
base: master
Are you sure you want to change the base?
Tring react #15
Conversation
…ssword = 12345
Registration work, write to base (login, pass) if not already registred
Now Login via autogenerated passwd via /login
With base auth work register via Postman post
If you login in browser and go to /username - it print username from Priincipal, but it don't work from postman with base auth
I think properties.setProperty("hibernate.hbm2ddl.auto", "update"); didn't work, can't add colum, if you change to "create", it make new table easy
Some debug option added/
TAking new version from hedin
# Conflicts: # src/main/java/ru/hedin/modelka/config/WebSecurityConfig.java
…ssword = 12345 Registration work, write to base (login, pass) if not already registred md5 work on registration and login Base Hold md5 password Troubles - see no reaction on roles
…ssword = 12345 Registration work, write to base (login, pass) if not already registred md5 work on registration and login Base Hold md5 password settings from security.xml work, not all can't make sucseclogin redirect/// All over work One strange think - registration can make only not logged Principal (Anonimous)- it is ok, by logic, but may be need to all ???
Trying rifsec fromdb
Need something about cross site
…ht cant get data. Headers is ok. Strange things - in headers only allowed GET - ignoring settings.
get auth - go java, but return 405
get auth - go java, but return 405
Tring react
get auth - go java, but cors
get auth - go java, but cors
…to Tring-react-ant
| <groupId>org.springframework.boot</groupId> | ||
| <artifactId>spring-boot-starter-web</artifactId> | ||
| </dependency> | ||
| <dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем нам httpclient?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Чтоб добавить хедер с количеством записей в json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Клиент это зверь который вызывает по хттн другие хосты. Он не должен быть для этого нужен.
| } | ||
| } | ||
|
|
||
| /* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
стоит убрать если это не нужно.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
не помню чтоб ставил, что ты хочешь убрать коменты или скобки ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
коменты
| // @Override | ||
| // @Bean | ||
|
|
||
| public void addCorsMappings(CorsRegistry registry) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А это нам сейчас нужно?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Нет наверное, если мы не вернемся к необходимости корс
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Не нужное давай убирать, если очень нонадобится поднимем в истории.
| import javax.servlet.http.HttpServletResponse; | ||
| import java.io.IOException; | ||
|
|
||
| /* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Лучше убрать закоментированное.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ну если убирать корс фильтр, то и все что в нем
| @EnableWebSecurity | ||
| @EnableGlobalAuthentication | ||
| @ComponentScan | ||
| @ImportResource("classpath:security.xml") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Надо бы проверить будет ли все работать если @ImportResource("classpath:security.xml") утащить на основной конфиг а этот совсем убрать.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это вобще другое, но надо проверю..
|
|
||
|
|
||
|
|
||
| /* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Дальше кажется все мусор.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Моджет быть, я в еррор не лазил, это ты делал
| // private BCryptPasswordEncoder passwordEncoder = new BCryptPasswordEncoder(12); | ||
| public boolean creteUser (ModUser modUser){ | ||
|
|
||
| // User to save to database via repository @link |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Старые варианты лучше убирать.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хорошо почищу
| // if (null == user) { | ||
| ModUser user = userRepository.getUserByLogin(username); | ||
| if (null == user) { | ||
| // throw new BadCredentialsException(""); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это было мое изменение чтобы поиграться во временной ветке.
Кажется оно не нужно в мастере.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ок, я не трогаю твой код
|
Ставь свои версии js, в моих состояние странное, и по скольку я так и не понял, что там с логином, и вид плохой |
No description provided.