Skip to content
This repository has been archived by the owner on Nov 23, 2022. It is now read-only.

Axum Rewrite #74

Merged
merged 150 commits into from
Dec 6, 2021
Merged

Axum Rewrite #74

merged 150 commits into from
Dec 6, 2021

Conversation

hydrostaticcog
Copy link
Member

@hydrostaticcog hydrostaticcog commented Nov 30, 2021 •
edited
Loading

As more patches are submitted to the axum-rewrite branch, we grow nearer to finishing. This PR will merge the branch into develop upon completion of the following methods:

  • Auth
  • Channels
  • Guilds
  • Roles
  • Invites
  • Members
  • Messages
  • Users
  • Bots
  • Information
    • WS
    • Ping
  • All the Errors

tazz4843 and others added 30 commits November 25, 2021 22:08
@tazz4843
basic work on the rewrite
4e48caa
@tazz4843
update auth struct to axum
9099922 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
add new crates needed for JSON and auth structs
85885d2 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
redo errors to always return JSON
cd6c676 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
do more changes
2a262df 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
more changes
f929e97 
this includes getting get_token.rs to work entirely

Signed-off-by: 0/0 <[email protected]>
@tazz4843
last changes to get framework working
90035c2 
this commit fixes entrypoint.rs and leaves it that way the only errors are in actual code functions

Signed-off-by: 0/0 <[email protected]>
@tazz4843
rewrite node ID conversion that I missed
f68cb24 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
rewrite node ID conversion that I missed (pt2)
e9f3717 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
rewrite send message
b8d1e33 
Signed-off-by: 0/0 <[email protected]>
@hydrostaticcog
rewrite get_message
83c4e09 
Signed-off-by: hydrostaticcog <[email protected]>
@tazz4843
fix typo in send_message.rs
f8fb8ec 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
Merge remote-tracking branch 'origin/axum-rewrite' into axum-rewrite
2896e9e
@tazz4843
rewrite create_channel.rs
0a8c1b4 
Signed-off-by: 0/0 <[email protected]>
@hydrostaticcog
rewrite get_member, fix get_message
7b44987 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
fix zero's oopsie
74bef66 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
add migration
56ad00f 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
rewrite get_invite, also fmt
800a31e 
Signed-off-by: hydrostaticcog <[email protected]>
@tazz4843
fix error in bigdecimal_to_u128!
095527c 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
Merge remote-tracking branch 'origin/axum-rewrite' into axum-rewrite
2ed49f8
@tazz4843
fix create_guild.rs
f6e75e5 
Signed-off-by: 0/0 <[email protected]>
@hydrostaticcog
fix stuff
0462fbe 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
fix get_messages and fmt
7710abf 
Signed-off-by: hydrostaticcog <[email protected]>
@tazz4843
fix create_role.rs
37f42fe 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
fix a bunch of warnings
ab78203 
Signed-off-by: 0/0 <[email protected]>
@hydrostaticcog
fix get_channel and fmt
92a86dd 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
fix more stuff and fmt
78897df 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
rewrite get_role, fmt
ed0cbef 
Signed-off-by: hydrostaticcog <[email protected]>
@hydrostaticcog
rewrite ws_info, fmt
32e20df 
Signed-off-by: hydrostaticcog <[email protected]>
@tazz4843
refactor get_token.rs to unify a statement into one
9e051d8 
Signed-off-by: 0/0 <[email protected]>
tazz4843 and others added 17 commits December 4, 2021 20:41
@tazz4843
actually allow the server to compile on BSD
67f29a5 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
update common repo
deaa08e 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
BREAKING CHANGE: update get_token to use only JSON body instead of he…
23916e5 
…aders for auth

Signed-off-by: 0/0 <[email protected]>
@tazz4843
fix all warnings except unused functions
d9fd67e 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
fix missing role functions
73eb0fc 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
fix missing verify functions
505ec25 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
remove actix from dependency tree
40e61b6 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
update runtime to tokio
306ff9e 
Signed-off-by: 0/0 <[email protected]>
@hydrostaticcog
Merge branch 'develop' into axum-rewrite
b7f33e4
@tazz4843
cargo fmt
51d304b 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
bump axum to 0.4
5dbfae7 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
fix errors from bumping axum to 0.4
8eedf11 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
add logging for all routes being made
04c97e6 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
fix panic on route creation with overlapping routes
4681da6 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
actually have the server shut down on ctrl+c
d9f232c 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
add logging in token verification
f102686 
Signed-off-by: 0/0 <[email protected]>
@tazz4843
use Display impl for u128
653fdb6 
Signed-off-by: 0/0 <[email protected]>
@hydrostaticcog
Copy link
Member Author

This PR is now awaiting QA testing

@hydrostaticcog hydrostaticcog marked this pull request as ready for review December 6, 2021 03:15
@hydrostaticcog hydrostaticcog requested a review from a team as a code owner December 6, 2021 03:15
tazz4843
tazz4843 approved these changes Dec 6, 2021
View reviewed changes
Copy link
Member

@tazz4843 tazz4843 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hydro approves

@tazz4843 tazz4843 merged commit 56a187a into develop Dec 6, 2021
@hydrostaticcog hydrostaticcog deleted the axum-rewrite branch December 6, 2021 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tazz4843 tazz4843 tazz4843 approved these changes

Assignees

@tazz4843 tazz4843

@hydrostaticcog hydrostaticcog

Labels
API Authentication enhancement New feature or request help wanted Extra attention is needed
Projects
No open projects
v1
Status: Completed
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hydrostaticcog @tazz4843 @jay3332 @Cryptex-github