omg.lol icon indicating copy to clipboard operation
omg.lol copied to clipboard

[Bug] Address name works in Account API

Open wayneyaoo opened this issue 2 years ago • 2 comments

Bug Description

I'm not completely sure if these two API should work (but they actually work).

https://api.omg.lol/account/wy-test/addresses
https://api.omg.lol/account/wy-test/sessions

because the accont/{stuff} is supposed to be an email address, not an address inside the account.

Given this

https://api.omg.lol/account/wy-test/info

and this

https://api.omg.lol/account/wy-test/name

don't work, the above two should not work right?

I understand there's nuance between the modeling of "account" and "address" and they're ever changing. Totally fine if we have a clear definition of both and correct reflection in API in the later overhaul.

Steps to Reproduce

No response

wayneyaoo avatar Feb 07 '23 13:02 wayneyaoo

Is this a bug, or a feature? :) Should be documented though if the latter

ejstreet avatar Feb 08 '23 00:02 ejstreet

Is this a bug, or a feature? :) Should be documented though if the latter

seems to a bug but let's wait for adam's confirmation.. since wy-test is not an email address, and not following the pattern

wayneyaoo avatar Feb 08 '23 00:02 wayneyaoo