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

Question: Site Search not taking into account Path Mapping #1358

Closed
zaryk opened this issue Nov 27, 2017 · 13 comments
Closed

Question: Site Search not taking into account Path Mapping #1358

zaryk opened this issue Nov 27, 2017 · 13 comments
Labels

Comments

@zaryk
Copy link

zaryk commented Nov 27, 2017

I am using FileSystem Index. Path Mapping works when searching from admin and Fess Search Page, but Path Mapping doesn't seem to work from Site Search. Are there any configurations that need to be setup to allow this?

This is what works from admin:

Regexp.
file:/E:/Websites/test.site.gov/Attachments/orders

Replacement
https://test.site.gov/Attachments/Order

EDIT:
Also, when clicking a link on Fess Search Page it should be a url to this path:

  • test.site.gov/Attachments/Order/785EECCE-D...

but the link takes me to

As mentioned in another post, this link should include /fts/go instead. When I manually add this to the url, it takes me to the http url, which isn't open to the public. If I add https, then it takes me to the correct location. It looks like the correct url is changed to the /go/ url after clicking the url.

@marevol
Copy link
Contributor

marevol commented Nov 27, 2017

/go link is an expected behavior to log a click action.
I think that rewriting Location response header is failed in your reverse proxy setting.

@zaryk
Copy link
Author

zaryk commented Nov 27, 2017

I will take a look, but the Location Response Header shouldn't affect the Path Mapping. At least for admin and Fess search page, this works. But doesn't work for the Site Search Javascript.

@marevol
Copy link
Contributor

marevol commented Nov 27, 2017

The response of /go returns url replaced by Path Mapping at Location response header.

@zaryk
Copy link
Author

zaryk commented Nov 27, 2017

But, does /go/ handle what shows in the list, before clicking it?

image

This is what Fess Search Page and admin looks like:

image

@marevol
Copy link
Contributor

marevol commented Nov 27, 2017

What is Process Type of Path Mapping?

@zaryk
Copy link
Author

zaryk commented Nov 27, 2017

Displaying.

@zaryk
Copy link
Author

zaryk commented Nov 29, 2017

Will this require a rebuild of fess, or is there something I can quickly change?

@marevol
Copy link
Contributor

marevol commented Nov 29, 2017

This needs to modify FSS JS, not Fess.
To generate a new JS file:

  1. Clone https://github.com/codelibs/fess-site-search.git
  2. Replace with site_path in ./fss/11.4/src/templates/fess-result.hbs
  3. Run on Docker: https://github.com/codelibs/fess-site-search#run-on-docker
  4. Generate JS file

@zaryk
Copy link
Author

zaryk commented Dec 5, 2017

docker doesn't install on my machine for some reason, or it is just taking too long. Any idea when this can be done in your timeline of things?

@zaryk
Copy link
Author

zaryk commented Dec 7, 2017

I noticed the issue has been closed, but FSS Generator doesn't seem to have the fix yet.

Can this section of the js be changed without changing anything else, and what would it need to be changed to?

<cite>'+u((p=null!=(p=r.site||(null!=n?n.site:n))?p:f,typeof p===c?p.call(l,{name:"site",hash:{},data:o}):p))+"</cite>

@marevol
Copy link
Contributor

marevol commented Dec 7, 2017

Not released yet...

@marevol
Copy link
Contributor

marevol commented Dec 17, 2017

Released.

@zaryk
Copy link
Author

zaryk commented Dec 28, 2017

Hey,

The actual url is still pointing to the local path.

image

yfujita added a commit to codelibs/fess-site-search that referenced this issue Dec 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants