Skip to content

Commit 605d11a

Browse files
authored
Merge pull request #18 from microsoftgraph/patch-17
Refactored error code
2 parents 33e1607 + 517555c commit 605d11a

File tree

3 files changed

+45
-6
lines changed

3 files changed

+45
-6
lines changed

demos/02-add-aad-auth/graph-tutorial/src/App.js

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -81,11 +81,24 @@ class App extends Component {
8181
await this.getUserProfile();
8282
}
8383
catch(err) {
84-
var errParts = err.split('|');
84+
var error = {};
85+
86+
if (typeof(err) === 'string') {
87+
var errParts = err.split('|');
88+
error = errParts.length > 1 ?
89+
{ message: errParts[1], debug: errParts[0] } :
90+
{ message: err };
91+
} else {
92+
error = {
93+
message: err.message,
94+
debug: JSON.stringify(err)
95+
};
96+
}
97+
8598
this.setState({
8699
isAuthenticated: false,
87100
user: {},
88-
error: { message: errParts[1], debug: errParts[0] }
101+
error: error
89102
});
90103
}
91104
}

demos/03-add-msgraph/graph-tutorial/src/App.js

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,11 +87,24 @@ class App extends Component {
8787
await this.getUserProfile();
8888
}
8989
catch(err) {
90-
var errParts = err.split('|');
90+
var error = {};
91+
92+
if (typeof(err) === 'string') {
93+
var errParts = err.split('|');
94+
error = errParts.length > 1 ?
95+
{ message: errParts[1], debug: errParts[0] } :
96+
{ message: err };
97+
} else {
98+
error = {
99+
message: err.message,
100+
debug: JSON.stringify(err)
101+
};
102+
}
103+
91104
this.setState({
92105
isAuthenticated: false,
93106
user: {},
94-
error: { message: errParts[1], debug: errParts[0] }
107+
error: error
95108
});
96109
}
97110
}

tutorial/04-add-aad-auth.md

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,11 +74,24 @@ async login() {
7474
await this.getUserProfile();
7575
}
7676
catch(err) {
77-
var errParts = err.split('|');
77+
var error = {};
78+
79+
if (typeof(err) === 'string') {
80+
var errParts = err.split('|');
81+
error = errParts.length > 1 ?
82+
{ message: errParts[1], debug: errParts[0] } :
83+
{ message: err };
84+
} else {
85+
error = {
86+
message: err.message,
87+
debug: JSON.stringify(err)
88+
};
89+
}
90+
7891
this.setState({
7992
isAuthenticated: false,
8093
user: {},
81-
error: { message: errParts[1], debug: errParts[0] }
94+
error: error
8295
});
8396
}
8497
}

0 commit comments

Comments
 (0)