Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Account management #164

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

wuriyanto48
Copy link

@wuriyanto48
Copy link
Author

@Neustradamus
Copy link
Member

@mattn, @mdosch: What do you think?

@mdosch
Copy link
Collaborator

mdosch commented Jan 8, 2024

I hope that I'll find some time within the next 2 months to do some tests. But as always in OSS, there is little time left between $dayjob and $familylife. ;)

@Neustradamus

This comment was marked as off-topic.

Copy link
Collaborator

@mdosch mdosch left a comment

Choose a reason for hiding this comment

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

Sorry, I added this questions some time ago but was not aware that only myself can see them as long as I do not submit the review.

}

// CreateAccount Creates a new account using the specified username, password and extra attributes
func (c *Client) CreateAccount(username string, password string, attributes map[string]string) error {
Copy link
Collaborator

Choose a reason for hiding this comment

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

How is this supposed to work? I connect with account A and use this *Client to register account B?
I tried this and after sending the IQ I waited for 10s and nothing was sent back from the server (yax.im in my case).

return err
}

// SearchAccount search information repositories on the Jabber network.
Copy link
Collaborator

Choose a reason for hiding this comment

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

What is such a service? Any xmpp server? I'd need to know a search service that's known to work for testing purposes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants