-
Notifications
You must be signed in to change notification settings - Fork 2
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
[임영서] 6주차 과제 - complete #58
base: main
Are you sure you want to change the base?
Conversation
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.
과제 리뷰 완료
연말에 과제하느라 수고 많으셨어요!
매번 빠지지 않고 과제해오시는 점이 너무 좋아요.
코멘트 확인하고 RDS 리소스만 삭제해주세요:)
메리 크리스마스🎄🎁
datasource: | ||
url: jdbc:mysql://gdsc-rds.c7ga6qgkwwb5.ap-northeast-2.rds.amazonaws.com:3306/study6 | ||
username: admin | ||
password: gdsccloud |
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.
헉....! RDS 리소스는 지금 지우셨을까요?
public access 허용에 비밀번호까지 업로드 되어 있으면 다른 사용자도 접근할 수 있어요...!
RDS 리소스를 지우지 않으셨다면 다른 사람들이 접근할 수 없도록 삭제 부탁드려요!
public ResponseEntity<Void> updateDB(BookForm form){ | ||
Book book = new Book(); | ||
book.setName(form.getName());//form에서 받아온 이름으로 책이름 설정 | ||
book.setReason(form.getReason()); | ||
rdsService.saveBook(book); | ||
return new ResponseEntity<>(HttpStatus.CREATED); |
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.
지금 여기서는 BookForm
을 통해서 데이터를 전송하고, Postman에서도 x-www-form-urlencoded 형식의 데이터 인코딩을 썼어요. 그런데 우리가 API를 통해서 JSON 데이터를 보내게 되면 HTTP의 Body에 해당 데이터를 담아 전송하기 때문에, updateDB(@RequestBody BookForm form)
처럼 RequestBody
라는 어노테이션이 필요해요.
좀 더 자세한 건 나중에 프로젝트에서 배우기로 하고, 6주차 과제 수고 많았어요:)
200 ok
이것저것 눌러보다가 다른 분들 과제 지우는 commit해서 다시 revert했습니다.....