ProjectMeshnet/nodeatlas

Placeholder "Can't change" edit text is submitted as part of the form

Closed this issue · 1 comments

After editing a node, I began to get this. @lukevers, I believe it has to do with the edit form, which apparently submits the email as part of the form. The option to edit email address will be changed in the API shortly, but the form is incorrect in doing this.

Edit: After more investigation, this only happens when trying to resend verification emails to nodes which have... apparently been edited, but are somehow in the verification queue?

sqlite> SELECT * FROM nodes_verify_queue;
643196083509272864|��|Test node edited|Can't change||||38.621162|-76.409912|257|0|1375025126
2013/07/27 21:39:52 nodeatlas.go:252: DEBUG Heartbeat
2013/07/27 21:39:54 verify.go:174: WARNING Could not send verification email to "Can't change": 501 5.1.3 Bad recipient address syntax
2013/07/27 21:49:52 nodeatlas.go:252: DEBUG Heartbeat
2013/07/27 21:49:53 verify.go:174: WARNING Could not send verification email to "Can't change": 501 5.1.3 Bad recipient address syntax
2013/07/27 21:59:52 nodeatlas.go:252: DEBUG Heartbeat
2013/07/27 21:59:54 verify.go:174: WARNING Could not send verification email to "Can't change": 501 5.1.3 Bad recipient address syntax
2013/07/27 22:09:52 nodeatlas.go:252: DEBUG Heartbeat
2013/07/27 22:09:54 verify.go:174: WARNING Could not send verification email to "Can't change": 501 5.1.3 Bad recipient address syntax
2013/07/27 22:19:52 nodeatlas.go:252: DEBUG Heartbeat
2013/07/27 22:19:54 verify.go:174: WARNING Could not send verification email to "Can't change": 501 5.1.3 Bad recipient address syntax
2013/07/27 22:29:52 nodeatlas.go:252: DEBUG Heartbeat
2013/07/27 22:29:53 verify.go:174: WARNING Could not send verification email to "Can't change": 501 5.1.3 Bad recipient address syntax

This has been fixed since the merge of f.edit in #140