Logowanie do aplikacji

Question

Hej,

Piszę aplikację, w której fronend jest pisany w Angularze a backend w Spring Boot, chcę się zalogować do mojej aplikacji, ale ta metoda, którą napisałam nie działa. Poniżej wrzucam kod i jeśli mogłabym prosić o podpowiedź co jest nie tak.

Kod w SpringBoot

@GetMapping("/users")
    public ResponseEntity<?> userLogin(@RequestBody User userData){
        User user= userRepository.findByLogin(userData.getLogin());
        if(user.getPassword().equals(userData.getPassword()))
            return ResponseEntity.ok(user);

        return (ResponseEntity<?>) ResponseEntity.internalServerError();
    }

Kod w Angularze

loginUser(user: User):Observable<Object>{
    console.log(user)
  return this.http.get(`${this.Url},user`);

  }

 

Answer 1

Generalnie nie wiem  dlaczego używasz GET- do logowania kogoś...
W tym celu wykorzystaj POST, jakieś błędy? @PostMapping i dodatkowo @
Nie widzę samej kontroli błędów, przez co nie wiesz  z jakim obiektem tak naprawde masz doczynienia, czy w debuerze wyłapywany jest REQUEST? Jeśli nie to, albo ścieżka się nie zgadza, albo  błąd jest w danych.
Jak wygląda ten ${this.Url} ? Pytam, gdyż oczekujesz mapowanie na /users, a ten url powinien być raczej tylko domeną.

Jeszcze czy nie masz @RequestMappingu ustawionego na kontrolerze wskazującego na 1 węzeł endpointu ?

Jeszcze zgodność typów widzę przesyłaśz Observable<Object> jako json, to przy okazji miej na uwadze warto by zrobić model w angolarze User...

Comments

W tej klasie mam jeszcze jedną funkcję, która służy do rejestracji użytkownika i jak dodaję kolejną funkcję z Post Mapping dostaję następujący błąd i dodałam Request Mapping

Caused by: java.lang.IllegalStateException: Ambiguous mapping. Cannot map 'userService' method
pl.serwer.UserService#userLogin(User)
to {POST [/users]}: There is already 'userService' bean method
pl.serwer.UserService#userAdd(User) mapped.

 ${this.Url} wygląda następująco http://localhost:8080/users

---

teraz mówi Ci o tym, że masz dwie takie same metody (na takim samym endpoincie http://localhost:8080/users tego samego rodzaju POST, bo tak być nie może ), zmień w takim razie endpoint metody addUser np dodając
@PosttMapping("/user") szczegolnie ze dodajesz uzytkownika  i niestety metoda oprócz tej samej metody http  i tego samego fragmentu zasobu mapowania posiada ten sam argument.   
http://localhost:8080/user/

Nie wiem jaki masz poziom języka angielskiego, ale bez tego będzie Ci cięzko. Caused by: java.lang.IllegalStateException: Ambiguous mapping
Wyraźnie masz napisane i możesz nawet skorzystać z google, debugej i informacje o błędach własnie po to są, żebyś była wstanie naprawiać problemy.

OSTATNIA sprawa, dlaczego przesyłasz 

this.http.get.(`${this.Url},user`); // jako jeden string?
// zauważ że całość objełaś jako 1 argument i to powoduje błąd 
powinnaś przesłać 
 return this.http.post.(`${this.Url}`, user);  
// uwaga na format user.. 

https://angular.io/api/common/http/HttpClient#post
Spójrz proszę na argumenty. 
 

post(url: string, body: any, options: { headers?: HttpHeaders | { [header: string]: 

Może być potrzeba dodania opcji, szczególnie jeśli nie posyłasz zgodności modelu User
 

const httpOptions = {
headers: new HttpHeaders({
 'Content-Type': 'application/json'
  })
}

i dodania opcji jako 3 parametr do funkcji 

---

Teraz działa. Bardzo dziękuję za pomoc :)

---

bardzo prosze, jakby były dalej pytania w tym zakresie to nie ma problemu