Skip to content

Conversation

@hitime1234
Copy link
Member

Moves docs from my radio to 2016-site

INSERT INTO myury.api_key_auth (key_string, typeid) VALUES ('ARANDOMSTRINGOFCHARACTERS', (SELECT typeid FROM l_action WHERE phpconstant = 'AUTH_APISUDO'));
```

[please choose a better key than 'ARANDOMSTRINGOFCHARACTERS']
Copy link
Member

Choose a reason for hiding this comment

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

square brackets is markdown syntax - and you should probably give an example of generating a random key like openssl rand -hex 32 - which should come before the SQL statements

Copy link
Member

Choose a reason for hiding this comment

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

bump

In Config.toml:

```
myradio_api = "https://{hostname}/api/v2"
Copy link
Member

Choose a reason for hiding this comment

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

this should probably just explain how to connect it to localhost, don't bother with https/another reverse proxy

Copy link
Member Author

@hitime1234 hitime1234 Sep 15, 2024

Choose a reason for hiding this comment

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

the problem I encoured is there no easy way of running it over https which is required by myradio apache server, without reverse proxying through a domain.
as it looks like you might need to gen some of your own certificates before you can use api
x509: cannot validate certificate
I'll see if I can make it work on localhost

Copy link
Member

Choose a reason for hiding this comment

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

if you go to http://localhost:7080/myradio/ rather than https://localhost:4443, it will work over http (if you have already set up a local instance, you need to change the port in Config:$base_url in src/MyRadio_Config.local.php)

hitime1234 and others added 4 commits September 15, 2024 21:34
@ashhhleyyy
Copy link
Member

also, just a note, but in the UI you can apply review suggestions all at once, rather than it creating a separate commit for each

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants