🐛 Fetch the documentation URL dynamically

[chipytux]

Pull Request Description

Title: Fetch docs_url dynamically from FastAPI app

Description:

This pull request introduces a change to the fastapi-cli library that fetches the docs_url value dynamically from the FastAPI application instead of using a hardcoded value.

Implementation Details:

The change involves modifying the _run function to fetch the docs_url value from the FastAPI application using the app.openapi_url attribute. This ensures that the library always uses the URL specified by the application itself.

Testing:

The updated function has been thoroughly tested to ensure it correctly retrieves the docs_url value from the FastAPI application and show the appropriate URL.

 ╭────────── FastAPI CLI - Development mode ───────────╮                        
 │                                                     │                        
 │  Serving at: http://127.0.0.1:9901                  │                        
 │                                                     │                        
 │  API docs: http://127.0.0.1:9901/any-other-path     │                        
 │                                                     │                        
 │  Running in development mode, for production use:   │                        
 │                                                     │                        
 │  fastapi run                                        │                        
 │                                                     │                        
 ╰─────────────────────────────────────────────────────╯   

[chipytux]

Do you think this is a good approach @tiangolo ? 🙏🏻

😉 @pedroimpulcetto

[chipytux]

@tiangolo I do not have permission to configure a label for the PR. https://github.com/fastapi/fastapi-cli/actions/runs/10408815023/job/28827030912?pr=24

[tobb10001]

I ran across the same bug today and I can confirm that the provided patch resolves the issue.

[github-actions]

This pull request has a merge conflict that needs to be resolved.

[YuriiMotov]

@chipytux , thank you for your interest and efforts!

This PR needs to be updated to reflect latest changes.
We also need to add some tests (including tests with root_path specified and specifying app path using entrypoint parameter).

Would you like to work on this?

[Infamous003]

Hi, this sounds interesting. Could you tell me more about what exactly needs to be done?

[pedroimpulcetto]

oh, ok, I'm able to handle these changes since @chipytux isn't aware of it

cc: @YuriiMotov

[YuriiMotov]

I meant we need to resolve the merge conflicts (merge main branch into the branch of this PR) and ensure it still works as expected.

The entrypoint option was added recently, we need to make sure it works correctly with these changes.
And, we need to add tests with edge cases like root_path.