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

Cannot match route when creating new RPC service #45

Open
michalbundyra opened this issue Jan 10, 2020 · 1 comment
Open

Cannot match route when creating new RPC service #45

michalbundyra opened this issue Jan 10, 2020 · 1 comment

Comments

@michalbundyra
Copy link
Member

With version 1.0.4 when you create a new RPC service the route is initially not matchable unless you put a forward slash preceding your route name. The problem is in the module.config.php

Replicate this by creating a new RPC service, in my case with the service name updateWeather and route to match as update-weather.

Below is generated module.config.php snippet. Note that the route is missing a preceding forward slash. By default trying to access /update-weather does not match a route, adding the forward slash makes the route valid.

Obviously I didn't add the slash when I should have however there was no indication I should and the REST services create the route for you with the slash. There should be validation that the route starts with a slash or one added for you.

            'sarah.rpc.update-weather' => array(
                'type' => 'Segment',
                'options' => array(
                    'route' => 'update-weather',
                    'defaults' => array(
                        'controller' => 'sarah\\V1\\Rpc\\UpdateWeather\\Controller',
                        'action' => 'updateWeather',
                    ),
                ),
            ),

Originally posted by @cookejames at zfcampus/zf-apigility-admin#231

@michalbundyra
Copy link
Member Author

@cookejames I had the same issue also with RPC sevices. The auto-generated code did not added the slash for the route


Originally posted by @Nguimjeu at zfcampus/zf-apigility-admin#231 (comment)

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

No branches or pull requests

1 participant