why would this 404 redirect not work?

Question

I'm using this code for my 404 redirect:

{exp:channel:entries channel="about-us" limit="1" require_entry="yes" {gv_param_disable_default}}
        {if no_results || last_segment != url_title }

        {redirect="404"}

        {/if}
<!--content here-->     
{/exp:channel:entries}  

Why wouldn't this work? It seems to stop an additional segment from being added to the url, but if I just change the last segment to an invalid path, it just goes to the blank index template for that channel.

Answer 1

The only thing you normally need for 404 with a require_entry="yes" and strict urls enabled is:

{if no_results}{redirect="404"}{/if}

Can you just try this on a clean template? As others have said, a broader context could help debugging this.

EDIT: if, as per your comment, you know you should only have two segments, then simply modify the code as follow to redirect to 404 if more segments are added

{if no_results OR segment_3 != ""}{redirect="404"}{/if}

Have a look at this little blogpost on 404 pages in ExpressionEngine for more detailed information.