reactjs - React Router *Not Found* rendered first before main routes -


i working on huge project , have authorization system logged in users can access routes while not found component, served on logged out users. problem is, when user marked logged in , has valid token , proceed page (e.g /account), component not found rendered first , second later component account rendered correctly.

i have async componentdidmount validate token , if it's valid, set state of islogged: true. on each route in switch checking the state of islogged , send corresponding component.

my async componentdidmount 

{/* username localstorage */}  async componentdidmount() {   if ((username != "") && (username != null)) {     const logged = await checkthetoken({'username': username}).then((result) => {       if(result.status == 202) {          this.setstate({islogged: true});          this.setstate({checking: false});       };     });   } }

this router (* supposing trying access /user page. notice use react-router, react-router-dom ^4.1.1* )

   if (this.state.checking) {         return null;    } else {    return (       <router>         <switch>             {/* not rendered first , not found rendered before falsly */}              {this.state.islogged && <route exact path="/user" component={() => (<user islogged={this.state.islogged} username={username} />)} />}              {/* rendered first , not found not rendering @ correctly */}              <route exact path="/test" component={() => (<test islogged={this.state.islogged} />)} />              <route component={() => (<notfound islogged={this.state.islogged} />)} />         </switch>      </router>    );

i think problem in {this.state.islogged && ...} because if try access /test component rendered correctly without not found rendering first.

also, tested lifecycle methods

i think you're right , issue comes {this.state.islogged && ...}

lets take step step.

first this.state.islogged falsy. mean <user islogged={this.state.islogged} username={username} /> not in reactrouter configuration.

we can guess reactrouter match default component (<route component={() => (<notfound islogged={this.state.islogged} />)} />) since /user not in configuration.

the actual behavior correct.

the fastest way archieve goal move token check child component :

async componentdidmount() {   if ((username != "") && (username != null)) {     const logged = await checkthetoken({'username': username}).then((result) => {       if(result.status !== 202) {         history.push('/not-found')       } else {         this.setstate({islogged: true});         this.setstate({checking: false});       };     });   } }

rendering function : 

render () {  if(!this.state.islogged) return <span />  return <div>...</div> }

the main problem aproach require authenticated components implement this.

you'll need factorize code service avoid multiple calls.

a 2nd approach fatorize proxy component check :

<router>     <switch>         <route component={authenticated}>            <route exact path="/user" component={() => (<user islogged={this.state.islogged} username={username} />)} />         </route>         <route exact path="/test" component={() => (<test islogged={this.state.islogged} />)} />         <route component={() => (<notfound islogged={this.state.islogged} />)} />     </switch>  </router>

this component 1 carry token check.

rendering function : 

render () {  if(!this.state.islogged) return <span />  return {this.props.children} }

main problem here can't share between component "this.state.islogged"

if want share , update mutiple component using global state, highly suggest give @ redux.

since redux can have hard learning curve, if need share single value, try things using observable pattern. (example : observable service, observable service + connector )

all code example not tested , here guide in right direction. in opinion there lot of way archieve want, sadly can tell why not work , hope gave enough clues find solution adapted use case.


Comments

Post a Comment

Popular posts from this blog

Is there a better way to structure post methods in Class Based Views -

i'm using class based views in django , never specify url in ajax calls , omit action parameter in form because know these requests go through post method of class based view. notice view - profile page make many requests depending on user updating on page - post method become convoluted , if/else mess example: def post(self, request, *args, **kwargs): # handle user changing available date if request.post.get('availabledate') != none: self.updateavailabledate(request) return jsonresponse({'result':'success'}) if request.post.get('newprojectname') != none: creative_user = creativeuserprofile.objects.get(id = request.user.id) project = project.create_project(creative_user, request.post['newprojectname']) project.save() return jsonresponse({'projectid': project.id }) if request.post.get('projectimage') != none , request.files['file']: proj...
Read more

performance - Why is XCHG reg, reg a 3 micro-op instruction on modern Intel architectures? -

i'm doing micro-optimization on performance critical part of code , came across sequence of instructions (in at&t syntax): add %rax, %rbx mov %rdx, %rax mov %rbx, %rdx i thought had use case xchg allow me shave instruction , write: add %rbx, %rax xchg %rax, %rdx however, dimay found agner fog's instruction tables , xchg 3 micro-op instruction 2 cycle latency on sandy bridge, ivy bridge, broadwell, haswell , skylake. 3 whole micro-ops , 2 cycles of latency! 3 micro-ops throws off 4-1-1-1 cadence , 2 cycle latency makes worse original in best case since last 2 instructions in original might execute in parallel. now... cpu might breaking instruction micro-ops equivalent to: mov %rax, %tmp mov %rdx, %rax mov %tmp, %rdx where tmp anonymous internal register , suppose last 2 micro-ops run in parallel latency 2 cycles. given register renaming occurs on these micro-architectures, though, doesn't make sense me done way. why wouldn't register renamer ...
Read more

c# - Asp.net web api : redirect unauthorized requst to forbidden page -

im trying redirect unauthorized request forbidden page instead i'm getting forbidden page in response body , how can fix ? here's startup class : app.createperowincontext(storecontext.create); app.createperowincontext<applicationusermanager>(applicationusermanager.create); app.createperowincontext<applicationsigninmanager>(applicationsigninmanager.create); app.usecookieauthentication(new cookieauthenticationoptions { authenticationtype = defaultauthenticationtypes.applicationcookie, expiretimespan = timespan.fromdays(30), }); app.useexternalsignincookie(defaultauthenticationtypes.externalcookie); app.usetwofactorsignincookie(defaultauthenticationtypes.twofactorcookie, timespan.fromminutes(5)); app.usetwofactorrememberbrowsercookie(defaultauthenticationtypes.twofactorrememberbrowsercookie); the method im trying reach : [httpget] [authorize(roles = "admin")] public string getcurrentusername() { return userman...
Read more