Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
nsjames committed Nov 26, 2018
1 parent 87f2b7f commit 7d69093
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/services/SocketService.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ const socketHandler = (socket) => {

// All authenticated api requests pass through the 'api' route.
socket.on('api', async request => {
if(!request.plugin) return socket.emit('api', {id:request.id, result:null});
if(!request.plugin || request.plugin.length > 35) return socket.emit('api', {id:request.id, result:null});
request.plugin = request.plugin.replace(/\s/g, "");

// 2 way authentication
Expand Down

2 comments on commit 7d69093

@tsingyee
Copy link

Choose a reason for hiding this comment

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

Hi @nsjames does the chrome app version Scatter still safe?

@nsjames
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The extension is deprecated and unmaintained. It should not be used anymore: https://medium.com/getscatter/the-blockchain-isnt-just-for-web-applications-silly-rabbit-926a4ea5ccd1

Please sign in to comment.