Allowing different HOST headers to be used in ng serve

Bug Report or Feature Request (mark with an x)
- [   ] bug report -> please search issues before submitting
- [ x ] feature request

Allow ng serve to accept requests with host headers that are different to the --host setting. This is needed when accessing the server via a reverse proxy.
Versions.
@angular/cli: 1.0.3
node: 6.10.1
os: linux x64
@angular/common: 4.1.2
@angular/compiler: 4.1.2
@angular/core: 4.1.2
@angular/forms: 4.1.2
@angular/http: 4.1.2
@angular/platform-browser: 4.1.2
@angular/platform-browser-dynamic: 4.1.2
@angular/router: 4.1.2
@angular/cli: 1.0.3
@angular/compiler-cli: 4.1.2
Linux Ubuntu 16.04.2 LTS
Repro steps.
ng new my-dream-app
cd my-dream-app/
ng serve
Configure reverse proxy (Apache/Nginx/HAProxy etc)
e.g. http://myapp.example.com reverse proxies to http://localhost:4200
Browse to http://myapp.example.com
Resulting page displays:
Invalid Host Header
The log given by the failure.
No error log is produced
Desired functionality.
I would like to see a command-line option that specifies a list of allowed HOST: header values.
Mention any other details that might be useful.
It is possible to work around this problem by editing the hosts file on your OS so that myapp.example.com points to an IP address that the server is listening on. This way, the 'expected' HOST: header is correct and ng serve is listening on the correct IP address.
However this is not an ideal solution as it requires editing the system-wide hosts file. This may not always be possible or desirable. It is common to allow for different HOST: headers to be used in server applications (e.g. Apache, Nginx etc) for virtual hosting.
I understand that ng serve is not meant for production purposes, so virtual hosting does not need to be a consideration, but sometimes developers also need to use a reverse proxy when working remotely rather than on their local machine (e.g. local machine is Windows and you don't want to run node locally).
After researching this issue more, I found that --host <IP_ADDRESS> was a common solution to this problem but no longer works due to a regression introduced in 1.0.1. due to stricter host name checking in WebPack. I see other issues related to it  ( #6070 ), but they talk about disabling host checking (bad) or configuring public hosts (Using --public in webpack-dev-server or --piblic-host in angular-cli v1.1.0.beta.1 or newer.)
I tried installing Angular-cli 1.1.0-rc.0 and using --public-host http://public.host.com and --host <IP_ADDRESS> or --host <local IP Addr> but it still returns the Invalid Host header error when I browse to http://public.host.com
In one of the many threads I have read on this in the last few days, someone mentioned that Django has a configuration to allow multiple hosts and indeed it does. But it's done as a part of the configuration of the app by definign an array of allowed hosts. Can something like this be done?

So I was looking around more. There is a veritable spiders web of issues pertaining to problems since the host checking was implemented in webpack-dev-server.
And I just noticed this https://github.com/webpack/webpack-dev-server/pull/899 which allows for passing in a list of allowed hosts in webpack-dev-server.
Can angular-cli somehow implement a similar list of allowed hosts and pass this on to webpack?

I have same problem

I have a similar issue, I am also using service workers so using ng serve with ssl so will need to check how this will work. I am no expert but happy to look at this with some guidance.

I too am happy to work on this (with guidance).

Looks like some work has already been done on this in v1.1.0-beta.1, will install and try this out.

I mentioned using v1.1.0-rc.0 which includes that commit.
What I am asking for is different. In that commit, one needs to --disable-host-check which is bad as it just circumvents the security measures that were put in place in webpack-dev-server.
Also in that commit is the --public-host option. Although this is tecnically an 'allowed host' it's not as flexible as having a configuration setting with an array of allowed hosts. There are multiple use cases where only allowing a single 'public' host may not be sufficient, for example, branding based on URL, or needing to used different reverse proxies for different locations.
As an aside, the --public-host was not working for me in v1.1.0.rc.0. I still need to use my hosts file or use --disable-host-check. I will double check though.

The --public-host option expects a hostname.  The use of a URL was most likely the cause of the issue observed.
Until webpack-dev-server adds support for configuring multiple hosts, there's not much that can done on that front, unfortunately.
Realistically, ng serve is meant to be a basic development server.  If a development setup requires a more exotic configuration, a custom server used in combination with ng build --watch may be more appropriate.
Also, there's nothing inherently bad about --disable-host-check.  It's a tool that can be quite useful if used appropriately.  Regardless of the use of the option, ng serve is not intended to be publicly accessible.

If I use a hostname instead of a URL for --public-host, I get the following error:
live-reload-client' must be a full URL.
I agree that this cannot be done until the multiple hosts is added to a webpack-dev-server release. It looks like it's being added in https://github.com/webpack/webpack-dev-server/pull/899 , but i'm not sure if/when that will be released.
Can this be revisited once the above is indeed released in webpack-dev-server?

Opps sorry @Tommyf you did mention v1.1.0-rc.0 my bad.
With regard to the full url, these combination of parameters worked for me
a) "ng serve --host host-ip-address --public-host http://host-ip-address"  ( had to add http:// )
b) "ng serve --host <IP_ADDRESS> --disable-host-check",
I am happy with the --disable-host-check flag for local development, but I can appreciate the need to secure it in larger development teams where a proper web server would most likely be used as clydin suggested.

@ryanper in that example, you have the same IP address used for --host and --public-host? If that's the case, then this is not the scenario I am talking about.
The scenario I am talking about is this:
ng serve --host local-ip-address --public-host http://public-ip-address
Where local-ip-address is on a private network and not accessibly directly from the web and you have a reverse proxy that maps the public ip to the local one. In the above instance, you only get Invalid Host header unless you add --disable-host-check also.
In this scenario, running the following
ng serve --host <IP_ADDRESS> --disable-host-check
will load the page, but the websocket for live updates will not connect as it tries to use http://localhost:4200 which is obviously not accessible from the net :)

Can you tell me what is this about?
When I run
ng serve --host <IP_ADDRESS> --disable-host-check
The option '--disable-host-check' is not registered with the serve command. Run `ng serve --help` for a list of supported options. appears. And it gives that same Invalid Host header. I tried to update package.json from 4.1.1 everything to 4.1.2 and the same "is not registered" comes just like before.

@Tommyf the scenario you're describing is really not recommended.  'ng serve' is not a production ready or security hardened server; and is not intended to be.
The settings being discussed are intended to enable the creation of internal production representative development environments.  Docker is quite popular for this.
@mikkoh85 the option will be available in version 1.1 of the CLI.  A release candidate is available currently.

@clydin thank you for this info, I guess I read it in a bit of hurry :)

@clydin I understand that this is not production for production use. The scenarios I talk of are all for development only. I can list out feasible development scenarios for this if needed.
I came across this problem as I was going through the basic tutorial for using angular-cli. I imagine I am not the only one running into this problem. Although I found workarounds (Hosts file, and now the --disable-host-checks option in the as-of-yet unreleased versions), I feel it would be good if this stuff would work more easily for people new to angular-cli who want to have a remote, non-publicall-routable development server without having to jump through hoops.
Also, could you please clarify the purpose of the --public-host option? I was under the impression that it was intended for this purpose, where the public host is different to the listening host, but if this is not a recommended scenario, what is the real purpose of that option?

I suppose "public" is somewhat of a misnomer.  However, it is more succinct than "host that will actually be used to access the application".  Also, as the need for these options (and their very existence) is fairly recent, documentation has not fully caught up yet.
A fairly common setup that would need to leverage the option is to encapsulate the CLI within a Docker container.  Sharing a preconfigured container within a development team has the potential vastly simplify team development workflows as well as on-boarding of new members.
More extensive setups leverage multi-container configurations allowing for API, DB, or reverse proxy servers.  These setups allow for a fairly comprehensive mimicking of a final production deployment and all within one local machine.
A typical workflow for ng serve is one of local development.  Running ng serve in combination with an IDE and a web browser; allowing for code changes to be quickly visualized, experimented upon, and adjusted in near realtime.

Shouldn't the "host that will actually be used to access the application" work without needing to use --ignore-host-check though? Surely it should be a valid host name, otherwise the option does not make sense?
I do understand what you are saying, I myself am just trying to do 'local' development, but for various reasons, I cannot have my dev environment on my local machine.

I am on Windows and don't wish to run node on Windows.
I switch between computers - desktop at home, laptop on the commute.

For these reasons, my dev server is not on my local machine, but I still want to use it in the same manner as if it were (with exception of it auto-launching a browser). The available options, especially the new ones, suggest that this should be possible without having to circumvent the security restrictions (Which, I agree, aren't real security holes in this kind of scenario, but forcing people to circumvent them can cause bad practices to form).
Is it OK to discuss like this in here? I'm wary of being too chatty in what is essentially an issue reporting forum.

@Tommyf You can read more in-depth about the underlying security issue here: https://github.com/webpack/webpack-dev-server/issues/887
Also you might want to look into using a Docker setup to encapsulate the dev dependencies (node, the cli, etc.).  There was a talk at this year's ng-conf (https://www.youtube.com/watch?v=socWfhPJptE).

I'm also having this issue after updating from 1.0.0 to 1.0.4, with Angular-CLI in a docker container, hosting to <IP_ADDRESS>, when trying to access the served page via the docker-machine ip address.

I use Docker in my workflow in order to develop in the same environment from dev to prod. I'm on Win 7 so I use Docker toolbox. Everything was working fine until angular-cli v1.0.0. The problem with this update from webpack-dev-server is that now the command ng serve -H <IP_ADDRESS> --environment=docker does not work anymore.
I saw a lot of issues on the web, and that blocks me from updating my components (angular-cli) to get newest features.
For now, it doesn't matter since there is no crucial feature that I'm awaiting, but I fear that soon, it will the case. I've read several workarounds. But every workaround I tried also has impacts on live-reload. So, for now, all I can do is force the angular-cli version 1.0.0, which is working fine.
Hopefully https://github.com/webpack/webpack.js.org/blob/master/content/configuration/dev-server.md#devserverallowedhosts will be available in webpack and angular-cli. Hopefully, my use case will be taken into account. I need to connect with a Docker container to localhost (from the container) which is by default the IP address of the container: http://<IP_ADDRESS>.
Any plans / news on it ?

Please add this flag to webpack:

this already exists: https://github.com/angular/angular-cli/pull/6173

I can see how this would be useful in development...

Configure reverse proxy (Apache/Nginx/HAProxy etc)

but I hope  you're not running ng serve in production.

Dear god now!
I need this to test my webapp from local network

this works like a charm:
ng serve --host <IP_ADDRESS> --disable-host-check

https://stackoverflow.com/a/44055209/6455043

if you are using webpack dev server and cannot do an ng eject
run
ng serve --host <IP_ADDRESS> -port4200 --disable-host-check

smh classic over-engineering
If you listen on localhost --disable-host-check should be default.
aka ng serve.
Do you trust localhost?
Don't tell me some attacker has infiltrated your network, gained access to your computer only to talk to the websocket of your angular dev server lol
But I also have a question, where do you make it permanent so it doesn't bug me every damn time I have some dev server behind nginx?

Please note that a DNS Rebinding attack allows a malicious website to access resources within the local network including localhost.  So it is not a matter of trusting localhost but rather trusting every website visited from the computer.

/etc/hosts takes precedence over /etc/resolv.conf


Option for longer alarm

I'd like an option for a longer alarm. I have young kids so my sleep pattern is often very screwed up. I've been able to sleep through lots of alarms very easily. I'd like to have an option for the alarm to just keep going until i turn it off. I understand that this can negatively impact battery life, but it's more important that I actually get up than have a long battery life.

+1 I'd love to see this in morpheuz since I use phone alarm for such feature


Cleanup Changes for Webpack 2.0

Cleanup Changes for Webpack 2.0

Coverage decreased (-0.05%) to 96.825% when pulling 9626ce2dc0c18874f52a271ceab1c01951c70857 on cleanup-webpack-20 into 628a13e7ebbf0e2ea6dd84df927058fec34e22e3 on master.


out_forward: do not use SO_LINGER on Windows

Which issue(s) this PR fixes:
Fixes #1968
What this PR does / why we need it:
There is an obscure bug in out_forward that prevents events from
being sent to a remote node. It occures only on Windows, and is
reproducable only when SSL/TLS is enabled.
The root cause is not fully investigated yet, but disabling
SO_LINGER is confirmed to fix the issue. So this implements it.
Docs Changes:
Not required
Release Note:
"out_forward: fix data loss when SSL/TLS is enabled on Windows "

Thanks!

This fix is included in Fluentd 1.5.0 according to https://github.com/fluent/fluentd/blob/master/CHANGELOG.md.
According to https://support.treasuredata.com/hc/en-us/articles/360001479187-The-td-agent-ChangeLog, td-agent (.msi package for Windows) 3.4.1 is still based on Fluentd 1.4.2.
=> When will an .msi package for Windows be available that includes this fix? (Say, newer than td-agent-3.4.1-0-x64.msi that is currently available on https://td-agent-package-browser.herokuapp.com/3/windows )

Next td-agent, 3.5.0, will be released late july or early august.
If you want to use newer version in msi package, gem update is one way.

Next td-agent, 3.5.0, will be released late july or early august.
If you want to use newer version in msi package, gem update is one way.

Thanks a lot for your fast response. The following update worked for me:
Download https://rubygems.org/downloads/fluentd-1.5.2-x64-mingw32.gem and install it with
fluent-gem install --force --local *.gem


Adding keystone.addLists method

In another (not in this repo) project I found it getting really repetitive repeating list keys in the schema exports and the index file (twice - for import, and for list creation).
This PR addresses that. Usage optional, so I only used it in the demo project (the meetup schema is quite complex and I find it a bit more readable in that project to see which lists are actually being created explicitly)

Good points @timleslie - I've been thinking about this.

Note: I'm pretty 🤷♂ about this PR getting in, I'd still like it to, but the comment below is mainly because I think it's an interesting conversation. TL;DR this falls under my category of "tone setting" where you see "pure helper, unnecessary" and I think there's value digging into that!

For me, especially for minor API like this (the implementation is trivial) the issue is not so much surface area, as what are we telling people to do with our API.
Adding a single list seems pretty useful (unless we decide to make list creation an atomic action, where you pass lists into the Keystone constructor, which I think has merit for debate but consider outside the scope of this conversation)
However, I don't think encouraging people to repeat themselves when it doesn't add value to the implementation is a good thing to do too much of. Explicitness is 💯 when there's a point to what you're being explicit about; but here (e.g in the Meetup demo) I just see unnecessary boilerplate & duplication.
For sure we could add that loop / function into the demo. But then those few lines of code will be repeated across hundreds and thousands of projects, adding no unique value each time. This seems like the sort of thing Keystone is good at removing from projects.
In most cases, I expect developers to define several lists across one or multiple files; import them; and register them. This API supports that with as little repetition and with minimal unnecessary expression in the project codebase - so from a "what is our API encouraging people to do" I think that's worth it.

because then where does it end?

Really good question, especially in the context of Keystone's history. I think it'll always be a trade-off, if we can get good at codifying how and why we make those trade-offs it would be awesome!
Keystone sits somewhere in the middle of the slider between "minimal framework API" and "minimal project code". Might be a good topic for a workshop to dig into this more and see what principles we can distill.
Real-world case study: our work on the CLI has come pretty directly out of trying to manage that balance - my original API was really quite verbose in the examples, which led to @jesstelford removing a lot of the non-differentiating code (read: boilerplate) through the CLI which in turn shrank the code required to set up a keystone project dramatically. The cost was more than one way to do things, and some confusion because the API was no longer as explicit. We've iterated through that to a really good place now, where there's minimal code required in each project but everyone feels it's a lot clearer what's going on (less "magic").
But even with the CLI we haven't really addressed the branching ways we've introduced to set up a keystone project, and probably should. Every quality of life feature we add introduces more complexity and we should probably get clearer on how we think about that and plan to manage it long term.

@JedWatson I also think that bing explicit is helpful for smaller projects. Writing createList() 4-5 times is better than hiding the names of the lists you're creating in different files.
For larger projects I imagine writing a for loop is a pretty quick and obvious decision, that is not much of a convenience method when you consider the scale of the project.
If you're ok I'd like to add this suggestion to the documentation and close this?


Composite resource problems RC 1.9.6

In testing for RC 1.9.6 I found the following problems (to fix before deploy) and quirks (not essential to address before deploy).
Problems

[x] Dragging a file into a folder results in an error
[x] If I have a zip file containing a tif file and attempt to set geo raster file type, I get the error below.  The tif file is OK as verified by setting the raster file after unzipping. [3/18/17 edit.  This is OK as an as yet unsupported use case.  This works if the zip file has a vrt file, but should also be made to work if it only has a tif file as a future enhancement.  @gantian127 please note as additional feature to implement.]

Quirks

[x] Unzip is only enabled upon right click
[x] Setting of a file type should create the file type in the current folder, not root level folder as it does now

The resource that illustrates these errors is https://beta.hydroshare.org/resource/074b845a667049719f96f34b660be906/
Error from dragging a file already in a resource into a folder

Error from setting geo raster on zip file containing tif file even when tif file is correct.

Unzip enabling illustration

@dtarb

Dragging a file into a folder results in an error

This problem is not specific to composite resource. It will happen with any resource type where such action is enabled. This is a bug with the recent work on REST API.
@aphelionz Would you look into it? The error comes as attribute error for accessing request.data ( https://github.com/hydroshare/hydroshare/blob/develop/hs_core/views/resource_folder_hierarchy.py#L363). There may be other places where request.data may have used to handle both rest call and UI call.

@dtarb

If I have a zip file containing a tif file and attempt to set geo raster file type, I get the error below. The tif file is OK as verified by setting the raster file after unzipping

It seems the validation of the vrt file is failing. @gantian127 can you comment on this? (https://github.com/hydroshare/hydroshare/blob/develop/hs_file_types/models/raster.py#L411)

@dtarb

Quirks
Unzip is only enabled upon right click
Setting of a file type should create the file type in the current folder, not root level folder as it does now

Both these will be fixed as part of the NetCDF PR.

@aphelionz, @pkdash - Has this been verified on www?

@mjstealey The following seems to work on www:

Unzip is enabled upon selection of zip file
Dragging a file into a folder works without the above error

Setting a file type creates a new folder in the current folder on beta but not on www. I am not sure why I am seeing this difference.

@mjstealey Ignore my comment related to file type setting and new folder creation. It works on www.

@dtarb Can you test and close this issue?

The last check box at the top is still not done.  Setting of a file type should create the file type in the current folder, not root level folder as it does now on www.

Done.  Close if tests out on next candidate release.

OK


apis_entities: fix merging two entities

The RDFParser needs the correct app_label_entities set, to find the
ContentType. We could also fix the RDFParser directly, but at the
current state this could have unknown consequences.
In the merge method of the TempEntityClass it seems that the lookup
of the ContentType also works withtout setting the app_label.

Skimming through the log I couldn't help but notice the entity.title() that's part of this change, which suspiciously looks like it could be a leftover case of referencing a class by title-casing its slug rather than using its actual model name (or its verbose name for display purposes), so thought I'd flag it up.
Issue summarising the problem which I've commented on in older PRs that fixed this issue in other places.


Check in the CI that pip install from Github work

This PR ensures that the regression in #17 won't recur.

On closer inspection this will require moving to a src/ layout for the package, so I'll revisit in a few days.


I was trying to compile on visual studio, but this error happened after trying to open the compiled .exe file.

Critical Error
There was an error while loading project data.
System.IO.DirectoryNotFoundException: Não foi possível localizar uma parte do caminho 'C:\Users\arthu\source\repos\Gibbed.Visceral\projects\Gibbed.Visceral.BigViewer\bin\Release\projects'.
em System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
em System.IO.FileSystemEnumerableIterator`1.CommonInit()
em System.IO.FileSystemEnumerableIterator1..ctor(String path, String originalUserPath, String searchPattern, SearchOption searchOption, SearchResultHandler1 resultHandler, Boolean checkHost)
em System.IO.Directory.GetFiles(String path, String searchPattern, SearchOption searchOption)
em Gibbed.Visceral.Setup.Manager.Load() na C:\Users\arthu\Source\Repos\Gibbed.Visceral\projects\Gibbed.Visceral.Setup\Manager.cs:line 92
em Gibbed.Visceral.BigViewer.Viewer.LoadProject() na C:\Users\arthu\Source\Repos\Gibbed.Visceral\projects\Gibbed.Visceral.BigViewer\Viewer.cs:line 50

If you're going to build on release you'll need to copy the directory bin/projects to the directory the executable is in named projects.


Slider suddenly loses structure and gets broken

Check out http://37.bdev.co.il/
There is a slick instance in the middle of the page with one navigation button [on the picture itself, you need to really look for it] if you click 7 times on it, everything gets broken...

Your container has display: table; on it, removing this seems to relieve/fix your problem. Sorry I don't believe you've created a sufficient bug report for further invesgitation.
Please refer to the contributing.md file in future.
Regards, Simon.

Sorry, didn't know display: table; can cause that. my bad.


chore(main): release bikeweek-backend 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-04-04)
Features

add version number to api surface (54474e7)

Bug Fixes

bump pino-pretty from 10.3.1 to 11.0.0 (d8dbcd3)
bump reflect-metadata from 0.2.1 to 0.2.2 (a398a63)
ecosystem config uses wrong startup path (45d1696)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/madisonbikes/bikeweek-backend/releases/tag/bikeweek-backend-v1.1.0 :sunflower:


Terra 2 upgrade

Whoa, appreciated.

Could you change the version of github.com/CosmWasm/wasmvm to v0.16.7 to support Apple Silicon yet?


[Doc]: xcdat API Reference not in (exact) alphabetical order

Describe your documentation update
I had a quick look at bounds' related stuff in the xcdat documentation, after reading this comment, and I used the Previous and Next buttons in the documentation.
I then noticed that the entries in the API index far from being in alphabetical order (see the out of place regridder in the bounds section, and other similar problems in the screenshot below). I don't know if this is a bug or a feature of readthedocs. Is there a way to fix this?

Thanks for pointing this out @jypeter! We'll take a look at this :)


Fix public CI w/ Numba 0.55

Run on Linux:
============================= test session starts ==============================
platform linux -- Python 3.8.12, pytest-6.2.4, py-1.10.0, pluggy-0.13.1
rootdir: /home/runner/work/numba-dppy/numba-dppy
collected 0 items / 1 error
==================================== ERRORS ====================================
________________________ ERROR collecting test session _________________________
/usr/share/miniconda/lib/python3.8/importlib/init.py:127: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
:1014: in _gcd_import
???
:991: in _find_and_load
???
:961: in _find_and_load_unlocked
???
:219: in _call_with_frames_removed
???
:1014: in _gcd_import
???
:991: in _find_and_load
???
:961: in _find_and_load_unlocked
???
:219: in _call_with_frames_removed
???
:1014: in _gcd_import
???
:991: in _find_and_load
???
:975: in _find_and_load_unlocked
???
:671: in _load_unlocked
???
:843: in exec_module
???
:219: in _call_with_frames_removed
???
/usr/share/miniconda/lib/python3.8/site-packages/numba_dppy/init.py:520: in
from numba_dppy.retarget import offload_to_sycl_device
/usr/share/miniconda/lib/python3.8/site-packages/numba_dppy/retarget.py:19: in
from numba.core.dispatcher import TargetConfig
E   ImportError: cannot import name 'TargetConfig' from 'numba.core.dispatcher' (/usr/share/miniconda/lib/python3.8/site-packages/numba/core/dispatcher.py)
=========================== short test summary info ============================
ERROR  - ImportError: cannot import name 'TargetConfig' from 'numba.core.disp...
!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
=============================== 1 error in 0.58s ===============================
Error: Process completed with exit code 2.

Fixed by pinning in https://github.com/IntelPython/numba-dppy/issues/592.
Support for the latest version is in https://github.com/IntelPython/numba-dppy/pull/585.


chore: sync workflows

Changes

chore: sync workflows (ece6d063fd1e69ce2c5f77424c289b710ef21e48)

Base PullRequest
default branch (https://github.com/technote-space/get-git-comment-action/tree/master)
Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

rm -rdf .github/workflows/.tmp

mkdir -p .github/workflows/.tmp

git clone --depth=1 https://github.com/technote-space/github-actions-workflows.git .github/workflows/.tmp/workflows
stderr:
Cloning into '.github/workflows/.tmp/workflows'...

bash .github/workflows/.tmp/workflows/gh-actions/copy.sh

sed -i 's/cron:.\+$/cron: 0 3 * * 3,6/' .github/workflows/update-dependencies.yml

rm -rdf .github/workflows/.tmp

Changed files

Changed file:

.github/workflows/broken-link-check.yml

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

rm -rdf .github/workflows/.tmp

mkdir -p .github/workflows/.tmp

git clone --depth=1 https://github.com/technote-space/github-actions-workflows.git .github/workflows/.tmp/workflows
stderr:
Cloning into '.github/workflows/.tmp/workflows'...

bash .github/workflows/.tmp/workflows/gh-actions/copy.sh

sed -i 's/cron:.\+$/cron: 0 3 * * 3,6/' .github/workflows/update-dependencies.yml

rm -rdf .github/workflows/.tmp

Changed files

Changed file:

.github/workflow-settings.json

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

rm -rdf .github/workflows/.tmp

mkdir -p .github/workflows/.tmp

git clone --depth=1 https://github.com/technote-space/github-actions-workflows.git .github/workflows/.tmp/workflows
stderr:
Cloning into '.github/workflows/.tmp/workflows'...

bash .github/workflows/.tmp/workflows/gh-actions/copy.sh

sed -i 's/cron:.\+$/cron: 0 3 * * 3,6/' .github/workflows/update-dependencies.yml

rm -rdf .github/workflows/.tmp

Changed files

Changed file:

.github/workflows/toc.yml

:octocat: Repo | :memo: Issues | :department_store: Marketplace


Override Styles in Subsequent CSS: text is blue but it should be pink

Challenge Override Styles in Subsequent CSS has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

Hello World!

As the result i can see blue text on the screen. But as we know - it should be pink, cuz the " the browser will use whichever CSS declaration came last."

In the exercise result must be blue - so it passed. I made a mistake and it gives pass through, but it shouldn't.

Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding.


OutOfMemoryError

Hi,
We are getting OutOfMemoryError very frequently while loading bitmap from KenBurnsView library. Here is the complete error logcat:
java.lang.OutOfMemoryError
at android.graphics.BitmapFactory.nativeDecodeAsset(BitmapFactory.java)
at android.graphics.BitmapFactory.decodeStream(BitmapFactory.java:677)
at android.graphics.BitmapFactory.decodeResourceStream(BitmapFactory.java:507)
at android.graphics.drawable.Drawable.createFromResourceStream(Drawable.java:872)
at android.content.res.Resources.loadDrawable(Resources.java:3054)
at android.content.res.Resources.getDrawable(Resources.java:1613)
at android.widget.ImageView.resolveUri(ImageView.java:648)
at android.widget.ImageView.setImageResource(ImageView.java:377)
at com.flaviofaria.kenburnsview.KenBurnsView.setImageResource(KenBurnsView.java:125)
Can you tell me the root cause? It seems that bitmap is not getting recycled as soon as the Activity killed by OS. Somehow bitmap remains in memory.
Thanks,
Sanjay

The images resolution is too large. I suggest you to <2000 pixel

Are you having this problem with the standard ImageView?
Em 23/06/2015 07:55, "Pagorn Phetchara"<EMAIL_ADDRESS>escreveu:

The images resolution is too large. I suggest you to <2000 pixel
—
Reply to this email directly or view it on GitHub
https://github.com/flavioarfaria/KenBurnsView/issues/27#issuecomment-114443910
.

Yeah, on standard ImageView too app crashes. This is the solution I found. All images were kept in drawable folder. So framework was applying its own scaling logic based on the device's resolution. Somehow the image wasn't recycled or the image size gets increased by auto-scaling logic applied by framework. When I moved all my images from drawable to drawable-nodpi, app never crashed.
Do you guys suggest any other solutions?
 I suggest you to use a library that handles image resizing for you in order to solve this issue. There are plenty out there, like Square's Picasso, Bump's Glide, Facebook's Fresco, Tarasevich's Universal Image Loader and Dutta's Ion. What you're having is framework specific, this is not a KenBurnsView bug.


docs: Add ByteDance as user

Checklist:

[x] My organization is added to USERS.md.

/cc @terrytangyuan


[BUG] Improvement Needed for Styling and Responsiveness in index.css

ADD LABELS: GSSOC EXT 2024, HACKTOBERFEST, POSTMAN
Describe the bug
The current styling in index.css does not adequately support responsive design, making the web application appear cluttered and difficult to navigate on smaller screens. Additionally, some elements lack sufficient visual appeal and accessibility features, which may negatively affect user experience.
To Reproduce
Steps to reproduce the behavior:

Go to the main page of the application.
Resize the browser window to a smaller width (e.g., below 768px).
Observe how the layout adjusts (or fails to adjust) to the new size.
Notice the appearance and usability of buttons and links.

Expected behavior
The layout should be responsive and adapt to different screen sizes without compromising usability. Buttons should be full-width on smaller screens, and the overall visual design should be improved for clarity and accessibility.
Screenshots
(If applicable, add screenshots to help explain your problem. Screenshots could show the cluttered layout or poorly styled elements on different devices.)
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 117.0.5938.88

Smartphone (please complete the following information):

Device: iPhone 12
OS: iOS 16.1
Browser: Safari
Version: 16.1

Additional context
The application is built using React and relies on the index.css file for styling. Improving the styles can enhance both user experience and accessibility. Suggestions include implementing media queries for better responsiveness, improving button styles, and ensuring links are easily identifiable and usable on touch devices.

For postman tag there is seperate repo provided by Gssoc there u will get postman tag on raising postman certificate PRs not here  @chikatlarakesh


Probably a bug in REST API gen

I have a REST method with params: out long totalCount, int skip = 0, int take = 100
I tried it with 0.7.26-alpha, but it failed to build on this assert:
https://github.com/rejectedsoftware/vibe.d/blob/master/source/vibe/web/rest.d#L716 (btw it could be more informative).
As it turned out it was a problem with the skip parameter marked as out.
I think that the problem is here:
https://github.com/rejectedsoftware/vibe.d/blob/v0.7.26-alpha.2/source/vibe/web/internal/rest/common.d#L233
there is used ParameterStorageClasses from all params to set in/out of single one.
I replaced
foreach (SC; ParameterStorageClassTuple!func) {
with
enum SC = ParameterStorageClassTuple!func[i];
and it seems to work, but I did not look at it properly

Accidentally committed to the "rest-refactor" branch, but that should hopefully be merged to master in one or two days. But I've fixed it exactly like you did above.


Make sure direct parameters are properly set on device

As pointed out in #1041, one dispatching a model that has direct parameters, there is a device mismatch. This PR should fix it by adding the main execution device to the hook attached to whole model.
Fixes #1041

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Info pages in footer added

Needs real data. Also, not quite sure where we want to pull the content from. I'm pretty sure we don't want to make separate templates for what are essentially identical pages just with different text.

Let me know what we need and I'll commit some more to this request.

Most of these are going to be place holders for now. Generally the terms is like we have permission to use your data or something.
The help page will fill in more as we build out functionality.
We could make a page schema if we wanted to use them in a eb. Or just put all their data in partial views.

Most of these are going to be place holders for now. Generally the terms is like we have permission to use your data or something.
The help page will fill in more as we build out functionality.
We could make a page schema if we wanted to use them in a eb. Or just put all their data in partial views.

Partials are easy. I'll work on that now.

Okay, we're good to pull this now. (We definitely need to change that filler text, haha)

I thought about making partials to drop into the footerpages.jade but ended up just making each one drop directly into layout.jade. There wasn't enough code to justify splitting it up so finely.


Generation of the SIRI classes

Hi,
I was wondering how you generated the SIRI classes. I tried it with JAXB, but had some issues with conflicting element names. Did you use xjc? If so, how did you invoke it? Could you maybe commit the generation XML/script/whatever into the repository?

First, thanks for your question! It's good to see that someone has used this.
I used xjc to compile, it has worked on Java6-8 on both OS X and Windows.
$ java -version
java version "1.8.0_45"
Java(TM) SE Runtime Environment (build 1.8.0_45-b14)
Java HotSpot(TM) 64-Bit Server VM (build 25.45-b02, mixed mode)
$ xjc -version
xjc 2.2.8-b130911.1802
$ xjc siriSg.xsd
parsing a schema...
compiling a schema...
eu/datex2/schema/_2_0rc1/_2_0/AbnormalTraffic.java
...

Sorry for not replying. Thank you for the command line! I had some troubles with some of the types (notably the ones defined ifopt/ which collide with the siri types). But I now managed to resolve them using rewrite rules in the bindings file.


[feature request] Ability to pass your own md parser

Would it be possible to update the component to be able to pass a custom md parser (similar to renderer, plugins)? I can create a PR as well if you want.

Thats a good idea, yea i think so. But what is the point? is it not easier to just create your own markdown component?

v2.0 now supports this


Strange bug with element.hover

I've come across a strange issue that has made my test suite unstable. element.hover occasionally fails to work and has a side effect.
After investigating various things, I inspected the element tag_name and text before calling element.hover and after. The element is changed to the html element after the call. This happens about 10% of the time.
Any insights into this behaviour? Those intermittent failing tests are driving me crazy.

Closing due to lack of response


added bid unit tests

Test with npm run test --findRelatedTests ./test/auction/auction.test.ts

https://github.com/tqtezos/minter/blob/53fcf5395657645a31b5fb429dd8307c2708ec41/contracts/test/auction/auction.test.ts#L37-L39
Consider originating contracts in beforeEach hook. This way contracts state (storage) from the previously run test is not interfering the current test

https://github.com/tqtezos/minter/blob/53fcf5395657645a31b5fb429dd8307c2708ec41/contracts/test/auction/auction.test.ts#L37-L39
Consider originating contracts in beforeEach hook. This way contracts state (storage) from the previously run test is not interfering the current test

Added this feature


Not showing anything when no AirPlay server available

@kswilster you think you could help a bit more on that? The latest version doesn't show anything anymore when there's no AirPlay server available, and your bash seems much better than mine 😅

I'll try to find some time to look into that tonight
On Thu, Mar 28, 2019, 6:52 AM Guill Lo<EMAIL_ADDRESS>wrote:

@kswilster https://github.com/kswilster you think you could help a bit
more on that? The latest version doesn't show anything anymore when there's
no AirPlay server available, and your bash seems much better than mine 😅
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/godbout/alfred-airplay/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCuQWjIrMNFan5dl37jxCWDVCoHQ6--ks5vbJ74gaJpZM4cP3QA
.

Ok no need, I've pushed a new version that handles that 😅
Thank you for the links in your PR, very helpful!


Review: ViewOrganizations.jsx and SearchOrganizations.jsx

Overview
The focus for this code review will be centered around the files in this project.
Please pay attention too:

Javascript issues
React components

Review Branch
review-11
Files to review

ViewOrganizations.jsx
SearchOrganizations.jsx

Checklists

Architecture
Design
JS
React
Testing

Due date
04/15
For more information
The review process is documented at: http://courses.ics.hawaii.edu/ics414s21/morea/review/reading-idpm-review.html

ViewOrganization.jsx
DE-05: line 64 remove chatgpt comment
JS-01: const is named VolunteerOrganizations instead of ViewOrganization

ViewOrganizations.jsx
JS-08 : document complex parts of the component
SearchOrganizations.jsx
No error find

ViewOrganization.jsx
Replace placeholder text with data from collection (mission statement, description, tags)
Implement the upcoming events section of the organization
SearchOrganizations.jsx
Implement categories of the organization

ViewOrganization.jsx
RE-08 : the file is named ViewOrganization, but its component is called VolunteerOrganizations
Replace the placeholder text with real data from collection
SearchOrganizations.jsx
no error founded

ViewOrganization.jsx
Nothing of importance to add
SearchOrganization.jsx
Nothing of importance to add

ViewOrganization.jsx
Rename VolunteerOrganization to ViewOrganization
SearchOrganization.jsx
No errors?

ViewOrganization.jsx
DE-05: line 64 remove comment


Add a TimeCache data structure.

[ ] Description
Add a TimeCache data structure.
[ ] Use cases

Save the world.
Stop the imposter.
....

[ ] Proposed design
A clear and concise description of the design planned for this feature/enhancement.
Attach design diagrams if applicable.
[ ] Alternative design approaches
A list of other approaches that could've been taken to design this feature/enhancement,
but hasn't been because of a specified reason.
[ ] Workaround
If any workaround exists to achieve the desired goal without this feature/enhancement.
[ ] Additional information
Add any other context about the problem here.

Won't add as it is already available in Guava.
TimedCache - https://github.com/google/guava/wiki/CachesExplained#timed-eviction


⚠ Bofrost.de has degraded performance

In e2d8623, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 742 ms

Resolved: Bofrost.de performance has improved in 9820ca4.


[12/n][eas-cli] Use new context for project ID

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
Same as https://github.com/expo/eas-cli/pull/1384 but for projectId.
How
Replace most of getProjectIdAsync with the new context param.
Test Plan
yarn test
Run all changed commands as a sanity check.

@wkozyra95 - Thanks for the review of this. That was the critical piece that was needed to refine this to make it work for all commands.
I updated this to encompass all config (getExpoConfig, getProjectId, and the dynamic versions of those two via a injected function). Let me know what you think.


refactor(msp): make use of the tower openapi client

This way we get the full tracing of the stack for pool operations
Remove seemingly unused crate dependencies

Thx for the cleanup with regards go unused dependencies🙏


Kernel update prevents ROCK64 from booting

Hello,
I recently installed the latest stable release  xenial-minimal-rock64-0.5.15-136-arm64.img.xz
on my Rock64. However it comes with kernel 4.4.77 but I need to upgrade it to any version equal or higher than 4.9 to be able to use Google BBR TCP congestion control algorithm.
I used the following reference https://github.com/ayufan-rock64/linux-build/blob/master/recipes/kernel-upgrade.md . With the command apt-cache search linux-image I chose the corresponding latest stable kernel available which is version 4.14 linux-image-4.14.0-rockchip-ayufan-136-gee91453 - Linux kernel, version 4.14.0-rockchip-ayufan-136-gee91453. After installing it with apt-get install <kernel>, I restart my system but unfortunately it doesn't boot anymore, the white LED on my board blinks twice constantly. What am I doing wrong? I really need to be able to use a higher kernel for my project.
I appreciate the help.

Hi,
I have exactly the same. Used different images with 4.4 kernel, followed the same procedure but after rebooting, seems to hang

Did you try the pre-release builds?  They have later kernels?
(I am unable to boot them either though .. maybe it's an issue with 4GB RAM boards?)

I just tested it works with the pre releases using xenial and bionic. Seems like an issue with the stable version..

I only tried the stable version. Will try the pre release this evenening


Upgrade libraries to latest patch releases

Upgrade scala compiler to 2.11.8
Upgrade Akka to 2.4.7
Upgrade all build plugins to their latest editions (which are mostly autoPlugins now)
Fix code formatting (commit with format changes next)

LGTM.
Btw IDEA doesn't really support scalariform yet (how sad), but a friendly person provided
a pre-commit hook.

👍


[FEATURE] DeviceType Support for SKView Touch Events to support stylus

SkiaSharp canvas view events always have touch device type even when touched with a pen.
SkCanvasView.OnTouch() events:
SKTouchEventArgs has a Touch device type which always registers as a “touch” event. on Android at
https://github.com/mono/SkiaSharp/blob/b8511b732b77b6e9f3b4929a5fa5598d189adf1d/source/SkiaSharp.Views.Forms/SkiaSharp.Views.Forms.Android/SKTouchHandler.cs
The event is being created using the constructor that doesn’t have the device ID :
var args = new SKTouchEventArgs(id, SKTouchAction.Pressed, coords, true);

Which is then defaulting to touch:
public SKTouchEventArgs(long id, SKTouchAction type, SKPoint location, bool inContact)
    : this(id, type, SKMouseButton.Left, SKTouchDeviceType.Touch, location, inContact, 0)
{
}

Describe the solution you'd like
in the SkTouchHandler.OnTouch, set the SkTouchEventArgs DeviceType based on the MotionEvent device type.
You can find the type under the motionevent property of the args then calling GetToolType on it
https://docs.microsoft.com/en-us/dotnet/api/android.views.motionevent.gettooltype?view=xamarin-android-sdk-9#Android_Views_MotionEvent_GetToolType_System_Int32_
Describe alternatives you've considered
none found
Additional context
none

This is a good idea to add. It looks like you already did some research, so you may have some ideas and code already. If so, just open a PR and we can talk code.
We don't have this feature yet just because I never did it. And nobody asked. But now you have 😄
That MotionEvent.GetToolType(int) member seems to be a API 14+ member, so you might just need to wrap it in a OS version check. But we can add that after we got the code actually working - just in case we have to adjust things. Or you can do it all in one go.
Anyways, from the looks of things, you should just be able to get the tool type and then use the overload that takes it.
If you are interested in writing the code, then just go ahead and try out things, open a PR and then you can add PR comments or start a discussion.
Oh, and thanks for the issue and the initial research.

This is now out on NuGet as 1.68.2-preview.50


[FLINK-9004][tests] Implement Jepsen tests to test job availability.

What is the purpose of the change
Use the Jepsen framework (https://github.com/jepsen-io/jepsen) to implement
tests that verify Flink's HA capabilities under real-world faults, such as
sudden TaskManager/JobManager termination, HDFS NameNode unavailability, network
partitions, etc. The Flink cluster under test is automatically deployed on YARN
(session & job mode) and Mesos.
Previous PR got closed accidentally: https://github.com/apache/flink/pull/6239
Brief change log

Implement Jepsen tests.

Verifying this change
This change added tests and can be verified as follows:

The changes themselves are tests.
Run Jepsen tests in docker containers.
Run unit tests with lein test

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no (at least not to Flink))
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no (but it will as soon as test failures appear) / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc: @tillrohrmann @cewood @zentol @aljoscha @yanghua

As @zentol mentioned, we might want to exclude this from create_source_release.sh.

Concerning @zentol's comment: We also include the flink-end-to-end-tests in the source release. Therefore, I think we could also include the jepsen-tests in the src release. Especially since one can execute them easily via docker-compose.

The jepsen license is not compatible with the apache license and thus must be part of the release.

Where exactly do you see the license conflicts @zentol? We do not include Jepsen code in our repository and since it is licensed under EPL 1.0 we should be able to link against it.

My information is outdated, they updated the license roughly a month ago to EPL:
https://github.com/jepsen-io/jepsen/commit/d9c8530b7a2deac2389d93639581dfa1ff0f08fc

I think you are right @zentol because we are linking against Jepsen 0.1.8 and Jepsen was licensed under EPL only in version 0.1.10.

Note that this may also imply that the files that we add cannot be apache licensed, but I'm not completely sure about that.

Gary is reaching out to Apache legal to ask for advice how to handle the SOLIPSISTIC license. Apparently, we cannot easily upgrade to 0.1.10 because it contains a bug.

0.1.9. was problematic. Currently testing 0.1.10.

It seems that we have to stick with 0.1.8 for now. According to legal, there are no concerns so far: https://issues.apache.org/jira/browse/LEGAL-392

I extended the regular README.md.

Nice, thank you. Merging this PR.


Report SwiftUI navigation and modal issues to Apple

As a demo user I want to be able to use the demo without navigation user experience issues like crashes or hangs. I currently experience the following issues in the demo, though:

Quickly navigating back with an edge gesture after dismissing a modal (e.g. returning to the show episode list, then back to the A-Z list) makes the app unresponsive.
Taps to display the player and modal dismissal made randomly in quick succession either:

Re-open the previous item, even if another one is tapped.
Sometimes crash the player because if duplicate presentation.

Hints
These issues seem entirely related to the use of .sheet for modal presentation and the use of NavigationStack for navigation.
Acceptance criteria

SwiftUI issues currently affecting the demo have been reported.
A mitigation strategy has been discussed.

Tasks

[x] Reproduce the issue with a small sample project.
[x] Report the issues separately to Apple.
[ ] Discuss the mitigation strategy to use .fullScreenCover instead of .sheet. Create a dedicated issue if accepted (close button and / or pull-to-close gesture have to be implemented in this case).

Edge gesture issue reported as FB12460230.

I made a quick implementation with .fullScreenCover. Though the result is stable we need a close button and pull-down gestures can be added but without interactive dismissal possible (the view behind the covering view is not there until full dismissal). So I guess we should keep .sheet for now.


TimeseriesGraph: additional attributes

For a marker we would have
{color?: string, shape?: 'circle' | 'square' | 'triangle, radius?: number}

For a line we would have
{color?: string, width?: number, dash?: [number, number]}

The "?" indicates optional.

This has now been implemented, but not released to pypi yet. Works from main branch.
Example:
https://github.com/magland/sortingview/blob/main/examples/example_timeseries_graph.py
https://figurl.org/f?v=gs://figurl/spikesortingview-9&d=sha1://f649d425bb2f3480034dbd8bfc10425157083513&label=Timeseries graph example
G.add_line_series(name='3', t=t.astype(np.float32), y=v.astype(np.float32), color='green', width=5, dash=[12, 8])
G.add_marker_series(name='4', t=t.astype(np.float32), y=v.astype(np.float32), color='black', radius=8, shape='square')


Timeout issue in dev mode with phoenix

I'm not sure if this is the correct place to report this issue.
When working with a phoenix live view app in development I get this error reported.
[error] GenServer #PID<0.1040.0> terminating
** (stop) time out
Last message: :timeout
State: {%ThousandIsland.Socket{socket:

I think this is related to development mode because it references the SyncCodeReloadPlug
plug: {Phoenix.Endpoint.SyncCodeReloadPlug, {WodWeb.Endpoint, []}}}, handler_module: Bandit.HTTP2.Handler, plug: {Phoenix.Endpoint.SyncCodeReloadPlug, {WodWeb.Endpoint, []}}}}

It may be something bespoke to my setup because I'm running HTTPS in dev.
Happy to provide more information or move to a different place if it's the wrong place to report

Have you tried the patch found in this ticket https://github.com/mtrudel/bandit/issues/74 ?

Thanks, have just tried and same issue

That timeout indicates the server timing out waiting to read data from the client. This could be the client being closed midway through an HTTP request, a network blip, or (99% of the time) related to a WebSocket client that's otherwise delayed client-side.
It's also possible that you've got timeouts configured abnormally. In normal operation, Phoenix's JS client will send a WebSocket heartbeat every 30s, which will normally arrive well before the default server timeout of 60s. If you've tinkered with your timeout settings (in your endpoint.ex file) that may be the cause here.
The SyncCodeReloadPlug is a red herring - that's a normal and expected part of the Plug stack in development (or any other environment where you have enabled automatic code reloading).

Interesting.  I've gone through and had a look, there's nothing changed with timeouts or heartbeat.
As far as I can tell, it's all basically vanilla. Only thing I think I have is that I run HTTPS in dev.
I can consistently reproduce the error be simply visiting any liveview page and waiting around 60 seconds.

I tried to reproduce this and have the same error, but only when using Firefox.
Did you also use Firefox? Did you test it with other browsers?

I do use Firefox.  Just tried in Brave, Safari and Chrome, can reproduce there too

Hmm, strange. In Safari I don't get the timeouts.
But I noticed a lot of "Could not determine protocol" errors from Safari "prefetching" the site:
https://user-images.githubusercontent.com/792046/211313653-fbd7b44e-379d-49ea-b8f3-874d6ee25903.mp4
@mtrudel
Maybe we should not error on this events and just ignore them? Or make it configurable? Otherwise the log get confusing fast.
Not sure how Cowboy handles these cases, but I did not notice these errors there before.

Is it only happening with https in dev mode?

Fixed in Thousand Island 0.5.15 / Bandit 0.6.5, both just released!

Thanks folks!


Error 500: Sync GitLab Remote - Fatal error: Uncaught PDOException

After click on the Sync Button for my GitLab Remote i got an Error 500.

Fatal error: Uncaught PDOException: SQLSTATE[23000]: Integrity constraint violation: 1451 Cannot delete or update a parent row: a foreign key constraint fails (devpackages.packages_gitlab_configurations, CONSTRAINT FK_1FE0005DF44CABFF FOREIGN KEY (package_id) REFERENCES packages (id)) in /var/www/vhosts/mydomain.de/httpdocs/packages/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:105 Stack trace: #0 /var/www/vhosts/mydomain.de/httpdocs/packages/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php(105): PDOStatement->execute(NULL) #1 /var/www/vhosts/mydomain.de/httpdocs/packages/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php(1006): Doctrine\DBAL\Driver\PDOStatement->execute() #2 /var/www/vhosts/mydomain.de/httpdocs/packages/vendor/doctrine/dbal/lib/Doctrine/DBAL/Connection.php(596): Doctrine\DBAL\Connection->executeUpdate('DELETE FROM pac...', Array, Array) #3 /var/www/vhosts/mydomain.de/htt in /var/www/vhosts/mydomain.de/httpdocs/packages/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/AbstractMySQLDriver.php on line 60

Thanks for the bug report! I thought this issue was already fixed, but apparently not!

can confirm the issue, with the same scenario, rename or remove a repository on gitlab break sync of the complete remote. should be fixed with prio I think, because it is a breaking bug and has a big impact.

Agreed, this is a pretty bad issue. It looks like it's going to require a database migration or improvements in how packages are deleted.
I'll take a look at what possible solutions we have as soon as possible.
As always, thanks for your contributions @strausmann and @NeoBlack ! It's very much appreciated.

I would recommend to add a delete flag to the model and mark the packages as "deleted", this make it possible to show this information also in the admin interface or frontend.
The information that a package is missing is very interesting, and a hard delete could be realized in the admin interface. anyway, the problem on delete is a wrong cascade definition in the database definition. Not sure where exactly, but the error message is really clear: " Integrity constraint violation"

Agreed on the soft delete implementation, @NeoBlack. Should have gone that way from the start. No matter what, we can handle whatever logic is necessary in a doctrine preRemove event subscriber.


Error downloading video: get_throttling_function_name: could not find match for multiple

Hey there, I'm getting this response when attempting to download a playlist. Prior to that, it was showing a certification issue, updated the 'Install Certification' code in python, and now I'm running into this. Any ideas?
Thank you

Updated the code and it works now. Also updated it to download past the 100-song limit and skip downloaded songs, changed the library to yt-dlp also:
import os
import urllib.parse
import urllib.request
import requests
import re
import string
from tkinter import Tk, ttk, filedialog, StringVar, Label
import yt_dlp as ytdlp
from dotenv import load_dotenv
import spotipy
from spotipy.oauth2 import SpotifyOAuth
from mutagen.mp3 import MP3
from mutagen.id3 import ID3, TIT2, TPE1, TALB, TDRC, TRCK, TCON, COMM, TPE2, TCOM, TPOS
Load environment variables
load_dotenv(dotenv_path='.env')
Setup Spotify API credentials
client_id = os.getenv("CLIENT_ID")
client_secret = os.getenv("CLIENT_SECRET")
redirect_uri = os.getenv("REDIRECT_URL")
Create Spotify client
sp = spotipy.Spotify(auth_manager=SpotifyOAuth(
client_id=client_id,
client_secret=client_secret,
redirect_uri=redirect_uri,
scope="user-library-read playlist-read-private playlist-read-collaborative"
))
playlists = {}
is_downloading = True
status_label = None
path_label = None
def sanitize_filename(filename):
valid_chars = "-_.() %s%s" % (string.ascii_letters, string.digits)
return ''.join(c for c in filename if c in valid_chars)
def update_playlist_dropdown():
playlist_names = list(playlists.keys())
playlist_menu = playlist_dropdown["menu"]
playlist_menu.delete(0, "end")
for name in playlist_names:
playlist_menu.add_command(label=name,
command=lambda value=name: selected_playlist.set(value))
if playlist_names:
selected_playlist.set(playlist_names[0])
def get_user_playlists():
global playlists
print("Retrieving user playlists...")
results = sp.current_user_playlists()
for item in results['items']:
playlists[item['name']] = item['id']
print("Playlists retrieved successfully.")
update_playlist_dropdown()
def stop_downloading():
global is_downloading
is_downloading = False
if status_label:
status_label.config(text="Downloading stopped.")
def get_playlist_tracks(playlist_id):
print(f"Retrieving tracks for playlist ID: {playlist_id}")
tracks = []
offset = 0
limit = 100  # Spotify API limit per request
while True:
    results = sp.playlist_tracks(playlist_id, offset=offset, limit=limit)
    if len(results['items']) == 0:
        break

    for item in results['items']:
        track = item['track']
        if track is not None:  # Check if track exists (some might be None due to availability issues)
            album = track['album']
            artists = [artist['name'] for artist in track['artists']]
            track_info = {
                "name": track['name'],
                "artists": artists,
                "album": album['name'],
                "album_artist": album['artists'][0]['name'],
                "release_date": album['release_date'],
                "track_number": track['track_number'],
                "disc_number": track['disc_number'],
                "total_tracks": album['total_tracks'],
            }
            tracks.append(track_info)

    offset += limit
    if len(results['items']) < limit:
        break

print(f"Retrieved {len(tracks)} tracks successfully.")
return tracks

def add_metadata(file_path, track_info):
audio = MP3(file_path, ID3=ID3)
# Remove existing ID3 tags
audio.delete()
audio.save()

# Add new ID3 tags
audio = ID3(file_path)
audio.add(TIT2(encoding=3, text=track_info["name"]))
audio.add(TPE1(encoding=3, text=track_info["artists"]))
audio.add(TALB(encoding=3, text=track_info["album"]))
audio.add(TDRC(encoding=3, text=track_info["release_date"]))
audio.add(TRCK(encoding=3, text=f"{track_info['track_number']}/{track_info['total_tracks']}"))
audio.add(TPOS(encoding=3, text=str(track_info["disc_number"])))
audio.add(TPE2(encoding=3, text=track_info["album_artist"]))
audio.save()

def download_songs(selected_playlist):
global is_downloading, status_label, path_label
is_downloading = True
if status_label:
status_label.config(text="Downloading...")
user_path = path_label.cget("text") if path_label else ""
download_folder = os.path.join(user_path, sanitize_filename(selected_playlist).replace(" ", "_"))
if not os.path.exists(download_folder):
    os.makedirs(download_folder)

playlist_id = playlists[selected_playlist]
tracks = get_playlist_tracks(playlist_id)

total_tracks = len(tracks)
downloaded_tracks = 0
skipped_tracks = 0

for index, track in enumerate(tracks, 1):
    if not is_downloading:
        print("Downloading stopped by user.")
        break

    sanitized_track_name = sanitize_filename(f"{track['artists'][0]} - {track['name']}")
    final_file = os.path.join(download_folder, f"{sanitized_track_name}.mp3")

    # Check if the file already exists
    if os.path.exists(final_file):
        print(f"Skipping, already downloaded: {final_file}")
        skipped_tracks += 1
        continue

    try:
        print(f"Processing {index}/{total_tracks}: {track['name']} by {', '.join(track['artists'])}...")
        search_query = urllib.parse.quote(f"{track['name']} {' '.join(track['artists'])} lyrics")
        html = urllib.request.urlopen(f"https://www.youtube.com/results?search_query={search_query}")
        video_ids = re.findall(r"watch\?v=(\S{11})", html.read().decode())

        for video_id in video_ids:
            try:
                url = f"https://youtube.com/watch?v={video_id}"

                # Get video info
                with ytdlp.YoutubeDL({'quiet': True}) as ydl:
                    video_info = ydl.extract_info(url, download=False)

                # Check if it's likely a lyric video
                title = video_info['title'].lower()
                description = video_info.get('description', '').lower()

                if ('lyric' in title or 'lyrics' in title or
                    'lyric' in description or 'lyrics' in description):

                    ydl_opts = {
                        'format': 'bestaudio/best',
                        'postprocessors': [{
                            'key': 'FFmpegExtractAudio',
                            'preferredcodec': 'mp3',
                            'preferredquality': '192',
                        }],
                        'outtmpl': os.path.join(download_folder, 'temp.%(ext)s'),
                        'noplaylist': True
                    }

                    with ytdlp.YoutubeDL(ydl_opts) as ydl:
                        ydl.download([url])

                    temp_file = os.path.join(download_folder, 'temp.mp3')
                    if os.path.exists(temp_file):
                        os.rename(temp_file, final_file)
                        add_metadata(final_file, track)
                        downloaded_tracks += 1
                        print(f"Downloaded and tagged successfully: {final_file}")
                        break
                    else:
                        print(f"Temporary file not found: {temp_file}")
                else:
                    print(f"Skipping non-lyric video: {title}")
                    continue

            except Exception as e:
                print(f"Error downloading video: {e}")
                continue
        else:
            print(f"No suitable lyric video found for {track['name']}")

    except Exception as e:
        print(f"Error processing track {track['name']}: {e}")

    if status_label:
        status_label.config(text=f"Downloaded: {downloaded_tracks}, Skipped: {skipped_tracks}, Total: {total_tracks}")

if status_label:
    status_label.config(text=f"Completed. Downloaded: {downloaded_tracks}, Skipped: {skipped_tracks}, Total: {total_tracks}")

print(f"Download completed. Downloaded: {downloaded_tracks}, Skipped: {skipped_tracks}, Total: {total_tracks}")

def select_path():
global path_label
path = filedialog.askdirectory()
if path_label:
path_label.config(text=path)
GUI setup
screen = Tk()
screen.title('Spotify Downloader')
screen.geometry("600x400")
Styling
style = ttk.Style(screen)
style.theme_use('clam')
Layout
frame = ttk.Frame(screen, padding="10")
frame.pack(fill='both', expand=True)
Path selection
path_label = Label(frame, text="Select Download Path:")
path_label.pack()
select_path_button = ttk.Button(frame, text="Browse", command=select_path)
select_path_button.pack()
Playlist dropdown
selected_playlist = StringVar()
playlist_dropdown = ttk.OptionMenu(frame, selected_playlist, "Loading playlists...")
playlist_dropdown.pack()
Fetch playlists and update dropdown
get_user_playlists()
Download button
download_button = ttk.Button(frame, text="Download", command=lambda: download_songs(selected_playlist.get()))
download_button.pack()
Stop Downloading button
stop_button = ttk.Button(frame, text="Stop Downloading", command=stop_downloading)
stop_button.pack()
Status label
status_label = Label(frame, text="")
status_label.pack()
Start GUI
screen.mainloop()

It's not working for me. What version of yt_dlp did you use?


Referencing attachment does not render properly

Currently, when referencing an attachment, it is showing the reference in plain text (see screenshot below).
Expected behaviour is to show the attachment as a clickable link, like in the web client.

This issue is related to #93 and will be fixed once this problem has been resolved. 9701 in the Artemis repository should be merged first.


Y coordinate of gene cluster is not changing accordingly when the aseq data is being opened.

Y coordinate of gene cluster is not changing accordingly when the aseq data is being opened. Need to fix this.
To reproduce: Select Genes/Protines in the main selection -> put query -> go to one of the genes -> click on DETAILS button -> mouse over genes in the cluster.

Resolved.


Power cycle buttons

This PR adds power cycle buttons for GS3xx series PoE switches. Their behavior is identical to the power cycle buttons in the web GUI.

Changes are testing on a GS308EP for 2 weeks.


Consider dropping import path support

It's not possible to determine ahead of time which VCS path for a given import path without adding more code.
It maybe simpler to not support checking import paths, and instead, only check the local directory and it's sub directories.

Are you familiar with vcs.RepoRootForImportPath?

I had trolled those repos looking for libraries to use/read, I can't recall why I didn't use them, will reinvestigate. But thanks for the reminder, I think it maybe useful in another related project.

No we're good on this one, give me a couple hours

TODO needs additional tests (see some discussion in #31, but mostly lack of coverage in new commits.


Validate Roles

If you can think of additional validations, please list them. We can release this for work once we have a good consensus on the validations we want.
Description
Roles have minimal validations, and some of them may not even be desirable. This is bad because we are currently vulnerable to developer mistakes or even malicious form manipulation to assign non-existent roles or other undesirable behavior. Authz are currently set to prevent such role problems from being an actual problem in regards to our data, but the roles themselves are vulnerable.
For example, if you try this in bin/rails c --sandbox:
> User.last.add_role(:wizard)
> User.last.roles
[#<Role:0x00000001112558c0
  id: 5,
  name: "staff",
  resource_type: "Organization",
  resource_id: 1,
  created_at: Thu, 28 Mar 2024 04:34:05.742340000 UTC +00:00,
  updated_at: Thu, 28 Mar 2024 04:34:05.742340000 UTC +00:00>,
 #<Role:0x00000001112557f8
  id: 6,
  name: "wizard",
  resource_type: nil,
  resource_id: nil,
  created_at: Thu, 28 Mar 2024 06:48:51.940040000 UTC +00:00,
  updated_at: Thu, 28 Mar 2024 06:48:51.940040000 UTC +00:00>]

We see two problems above. One, wizards don't exist in this app. Two, the role validations currently permit resources to be nil (which means any checks that forget to check for resource scoping would pass).
Note: The acceptance criteria includes some validations that may already exist, but we should confirm that rolify is enforcing them as we expect or else implement ourselves.
Acceptance criteria
Add validations for these to models/role:

[x] Duplicate roles with the same name and resource cannot be created (I think this is implemented by rolify's default, but we need to confirm so)
[x] Limit roles to the following: [:adopter, :fosterer, :staff, :admin] (inclusion validation will be useful)
[ ] Resource (organization) must match User's organization
[x] Name cannot be nil (this may be covered by above inclusion validation)
[ ] Resource type cannot be nil
[ ] Resource cannot be nil

Others?

[ ] User cannot have duplicate roles (think rolify also covers this but confirm)

Putting this on hold for now. I did start this and have three of the acceptance criteria completed in this branch. We might be able to start from it when this issue is necessary. https://github.com/mononoken/pet-rescue/tree/603_validate_roles

These validations can wait until after #679 is complete.

@mononoken  Is there any use case where a User would have multiple roles? I don't see it being applicable it our the current implementation, however in factories>users.rb it builds a user with both staff and admin roles. Currently implementing role changes #615


[style]: The background color in the dark mode can be enhanced from simple black to some different color gradient

What's the style idea?
Aim

The idea is that when dark mode is enabled simple black color appears. We could change that background color to some different color gradient which would look more beautiful.

Please assign me this issue under GSSoC-2023
Add screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

cool I'm also thinking like this. you can work on it.

ig we don't change the bg colour so I'm closing this PR


Use solidity filename for truffle

fixes #356

hey @rocky can you check whether this commit fixes #356 or not?


3: Backfill tests for OTLP Resource Attributes

Add test to validate this logic:
If the Resource and Span both have an Attribute with the same name, the Attribute on the Span is the one that will be used in the resulting Wavefront Span – the value of the Resource's Attribute will not be used.

I'm merging this:

#4 is merged into this branch
ESO does not need to QA Validate


Setup Precommit Hooks for Git

User Story:
As a software developer, I want my linting and test cases to run when I commit so that I can be sure that my build works and the style rules are followed.
Acceptance Criteria:

On commit, to Github the prettier and eslint rules are applied
Tests will also be run on commit

@res550 This was done in the frontend when I set up linting. See Issue #31 .

Okay cool ill just put the unit tests in then

Going to close this after discussing further it doesn't seem valuable as it just stop developers from using TDD if they choose to do so since you cant commit until all tests pass


Add a healthcheck instead of a post-start hook

Rather than a post-start hook, add a healthcheck like https://github.com/ddev/ddev-drupal9-solr/blob/008973f804e2a7e86a9e54533a2518b2a9be189a/docker-compose.solr.yaml#L77-L78
That will make it so that DDEV waits for it to become healthy on start.

Hmm, will the solr container health check result in DDEV waiting before running project-related hooks?

If the solr container has a healthcheck that does the right thing, ddev start will wait until all containers are "ready" before running any post-start hooks.


Use same contract address across all networks

It would be helpful to use the same contract address for deployed contracts across all networks.
This can be useful for testing purposes and helpful to users interacting with Semaphore addresses across various networks.
For reference: https://docs.alchemy.com/docs/how-to-deploy-a-contract-to-the-same-address-on-multiple-networks

Thank you!

Is this going to be reflected on: https://semaphore.appliedzkp.org/docs/deployed-contracts

@atomniketh It's still on the dev branch, we are testing everything before releasing a stable version 👍🏽


Current clues overlap with the grid on mobile phones when they span more than two lines

I am very pleased with the new (v80.0) responsive behaviour for mobiles. However, having just checked a few puzzles on my phone, I have found this problem (from my Alphabetical Jigsaw puzzle):

I also checked the mobile device toolbar in Chrome's (Windows) dev tools and found it shows the same issue when using one of the built in devices, the Galaxy S5:

Fixed now, with v0.81. PTAL, thanks.

Thanks! I've only tried it on my private site so far and it's a great improvement, but I notice it now overlaps the preamble instead. This is much less of a nuisance of course, as tapping a black square or refreshing the page will hide the clue strip if one wants to reread the preamble, but it seems to me like the strip should be dynamically adjusting the space for itself rather than overlapping anything. Although I imagine that is easier said than done. :^)

With any solution to this, there will be some threshold beyond which things will look bad. In the solution that I have put in place currently, if the clue is so long that it overshoots the top of the window itself, it will get clipped.
If we reduce the font size for long clues and stick to two lines, I think there will be a much smaller length threshold where the clue will be unreadable, especially on smaller devices. Otoh, if by "the strip should be dynamically adjusting the space for itself" you mean that the strip should just expand more and push the grid down, I'm afraid I'm not a fan of the idea of the grid jumping around (that would also mean that after tapping on a cell to enter a letter, the cell and the cursor might slip away to a location further below).
I can also try to estimate the space needed (based upon the longest clue length) and allocate that ahead of time, but that will be inexact and subject to device/platform vagaries. Moreover, if only a few clues are long, it will look like a lot of wasted space most of the time.
So, in summary, I cannot think of a more satisfactory solution for now. But am happy to consider more ideas, always! :-)

Fair enough. It's a lot better now that it was before, so I'm happy to declare this issue resolved. Thanks again.

Just one small follow-up: you can always place a few  tags at the end
of the preamble to create more blank space and avoid the overlap.
Viresh
On Sat, Aug 1, 2020, 7:50 AM Antagony1060<EMAIL_ADDRESS>wrote:

Fair enough. It's a lot better now that it was before, so I'm happy to
declare this issue resolved. Thanks again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/viresh-ratnakar/exolve/issues/32#issuecomment-667543167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJQ562B64WFJDZBYEAZBNTTR6QTSBANCNFSM4PN3MCWA
.

Interesting. I knew the preamble in my alphabet puzzle already had a few breaks, so I experimented with double length clues and found I could accommodate them by adding breaks and hard spaces (<br> ) as required to the end of the preamble. It shouldn't be necessary under normal circumstances, but it's useful to know if a puzzle ever has crazy long clues any time.


[serverless] Try encrypting KMS key without encryption context first

Note: This branch is based on #8519 and should not be reviewed until that PR is merged
What does this PR do?
When the API key is encrypted using the AWS console, the function name is added as an encryption context. When the API key is encrypted using the AWS CLI, no encryption context is added. We need to try decrypting the API key both with and without the encryption context, because decrypting using the incorrect encryption context will cause decryption to fail.
Previously, we tried decrypting WITH the encryption context first, then fell back to encrypting WITHOUT the encryption context. This PR reverses the order and tries decrypting WITHOUT the encryption context first, falling back to encrypting WITH the encryption context.
Motivation
We want to encourage customers to encrypt their keys using the AWS CLI, without the encryption context. That way, a single encrypted key can be re-used for multiple functions. This is necessary in order to use our deployment tools (e.g. Serverless Framework plugin).
Describe how to test your changes
This functionality is covered by existing unit tests.
Checklist

[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] The appropriate team/.. label has been applied, if known.
[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.

Closing this and opening another PR off of the main branch, as we don't actually need the refactor here.


improving new environment creation process

i have same env folder:
%anaconda%\env\py27
%anaconda%\env\py35
but when i add new one it always starts in "This PC" ... and i open dialog, navigate to my new env folder, then have to auto detect and go through all menus and select new environment folder... so much work! why not set default folder for new environments to last folder used, or even better auto-detect a new environment by searching the %anaconda%\env ...

We'd love to have special support for Anaconda here, but haven't had the time to make it happen.
Remembering the previous folder when browsing should be fairly straightforward here, though we do share a decent chunk of this code throughout the product, so there'll be more flow-on effects that may be expected.

We auto detect conda environments now :)


Add version number to Settings popup

Feature Request
What problem does it solve?
It will allow me to find what version I have installed.
Impact on the interface
Could add it to the bottom of the Settings pane. I'm sure it's somewhere else already currently, but I can't find it in an intuitive spot; this seems like a good intuitive spot to add the release version number.
An alternative would be in an "About" menu. I just found the "About" menu in the bar at the top. That might actually be good enough.

@DeflateAwning feel free to close the issue, if it is not needed anymore.


[Feature] add support svn

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
support svn
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for your feedback, currently, no one is driving this feature. If you are interested in contributing to this feature, we will accept this feature

Currently, there is no plan to support this feature.


unSupported: ltm monitor postgresql /Common/int-postgres-monitor

Environment

Application Services Version: ACC 1.15
BIG-IP Version: 15.1

Summary
While trying to convert a virtual server using a custom monitor for postgresql, I am getting an unsupported object error from ACC. However, I am not sure what is exactly unsupported. Is it the entire monitor? or a specific attribute?
Steps To Reproduce
Steps to reproduce the behavior:

Convert following config:

ltm virtual /Common/dc1_dev_corp_int_web_app_backend_vs {
    creation-time 2021-08-23:05:43:55
    description "F5-34532 - DEV"
    destination /Common/<IP_ADDRESS>:8080
    ip-protocol tcp
    last-modified-time 2021-09-16:15:06:00
    mask <IP_ADDRESS>
    pool /Common/dc1_dev_corp_int_app_pool_backend
    profiles {
        /Common/tcp { }
    }
    serverssl-use-sni disabled
    source <IP_ADDRESS>/0
    source-address-translation {
        type automap
    }
    translate-address enabled
    translate-port enabled
}
ltm pool /Common/dc1_dev_corp_int_app_pool_backend {
    members {
        /Common/int_apps.example.com:5432 {
            address <IP_ADDRESS>
            description F5-12346
            session user-disabled
        }
    }
    monitor /Common/int-postgres-monitor
}
ltm monitor postgresql /Common/int-postgres-monitor {
    count 0
    database pgdb_db
    debug no
    defaults-from /Common/postgresql
    destination *:*
    interval 5
    recv f
    send "select pg_is_in_recovery()"
    time-until-up 0
    timeout 10
}

Observe the following error message:

unSupported: {
    'ltm monitor postgresql /Common/int-postgres-monitor': {
      count: '0',
      database: 'pgdb_db',
      debug: 'no',
      destination: '*:*',
      interval: '5',
      recv: 'f',
      send: '"select pg_is_in_recovery()"',
      'time-until-up': '0',
      timeout: '10'
    }
  },

Expected Behavior
Convert my postgresql monitor to AS3 successfully.
Actual Behavior
See error above returned by ACC.

Thank you this is being tracked internally as CHARON-472


Admin name in sidebar

Do i need add an username for my admin users to display it in "sidebar__user" div ?

No, you can skip this div


Update configuration.md to include config for secure etcd

We are running etcd on DigitalOcean. So, we need to secure etcd endpoints via tls auth. Here is the pr https://github.com/projectcalico/calico-cni/pull/113/files . Does the new Go version support connecting to secure etcd ?
I don't see anything here: https://github.com/projectcalico/calico-cni/blob/master/configuration.md#etcd-location

@TamalSaha - Yes, it does.  That documentation needs to be updated!
See here: https://github.com/projectcalico/calico-cni/blob/master/utils/types.go#L65

Awesome, @caseydavenport

I've now updated the docs - see #158


Purpose of initial knitting step

The render_full_report function initially knitted the report before rendering it, as two steps.  This was successful in generating reports, but broke a lot of the formatting, particularly around figures and captions, so I switched it to only a single render step.
Do you remember the particular advantage of knitting the report as a separate step (was it to speed up rendering to multiple formats?) and/or how to fix the formatting issues?
I don't necessarily expect a fix here, feel free to just comment/reply to this issue if that's easiest.

@stoyelq I remember why!
I did this so that at the end of the process, the Rmd file includes all the text. Currently, with your changes the Rmd downloaded (when clicking on "Download outputs") cannot be used as the child Rmd documents are not included in the zip (they were in the previous version). I guess I thought it would make more sense to have everything in one document.
I think there are 3 options:

Remove the Rmd document
Add all child Rmd documents in the zip file
Try to go back to what I did and try to figure out why it does not work (a simple option could be to do an extra rendering as a Markdown document, which is basically what I was trying to do)

Yes, it makes sense to avoid having child sections in the output RMD!  Removing the rmd from the download is probably best.  Between the search area and the html in the output file, reproducing the report should not be a problem.

@stoyelq, just to be sure we're on the same page. I let the current render process as it is, and I simply remove the Rmd from the set of outputs to be downloaded.

@KevCaz, yes, that is correct.


Current docker image with Mariadb doesn't support 4byte unicode

The mysql-init.sql file creates the database with:
CREATE DATABASE vitrodb CHARACTER SET utf8;
This does not allow extended unicode to be uploaded.
Researching this issue it looks like the DB should be created with utf8mb4.
I've attached the rdf that I tried to update via siteAdmin
testNode.txt

@delsborg : Would you be interested in putting forward a pull-request?

Hi Andrew, see the utf8mb4 branch.
Also, it still doesn't work for uploads. See email I sent to tech group.
This docker image does help to recreate the problem in a manner that anybody can reproduce :)

Reference: https://github.com/vivo-community/vivo-docker2/tree/utf8mb4

@delsborg I added this into my PR for you if you didn't see:
CREATE DATABASE vitrodb CHARACTER SET utf8mb4 COLLATE utf8mb4_unicode_ci;


Updating Tests For Namespace Recommendations

Description
This PR updates the tests for namespace recommendations with updated format.
Type of change

[ ] Bug fix
[ ] New feature
[ ] Docs update
[ ] Breaking change (What changes might users need to make in their application due to this PR?)
[ ] Requires DB changes

Test Configuration

Kubernetes clusters tested on: kruize-lm

Checklist :dart:

[x] Followed coding guidelines
[x] Comments added
[ ] Dependent changes merged
[ ] Documentation updated
[x] Tests added or updated

Additional information
quay.io/rh-ee-shesaxen/autotune:ns-extype-change

@shekhar316 Please fix the conflicts

@shekhar316 Please fix the conflicts

Rebased the PR.


Scan ID slicing

By popular demand, fetch multiple scan IDs with slicing syntax. db[2:5] gets the scan IDs in range(2, 5), which is 2, 3, and 4.
This improves test coverage of the various ValueErrors that catch inputs that are illegal ([-2:5]) or unallowably dangerous ([3:]).
Implementation Details / Design Rationale:

As usual, where the scan ID is ambiguous, the most recent one is used. This will probably lead to someone someday accidentally getting old scans, but this is the best we can do.
One might expect pandas-like label-based slicing, where [2:3] return 2, 3, 4 -- not 2, 3 as in standard Python slicing. But (1) this is a sharp edge of pandas, perhaps best not imitated, and (2) users seems to view scan ID as an auto-incrementing counter, not a "label" per se.

Note to self: needs a confusing rebase

@danielballan what is the status of this?

Tom hates it, and I am +0.


Update Notice for !ask command along with slight fix

@Coolguy3289 Reverted replacement of icon. In future I will relocate all icons with a few test


THRIFT-4996 python3 modules are not installed

Client: Python
Patch: Paulo Neves
Submitted on behalf of Paulo, see JIRA ticket.

The AppVeyor build failed.
Creating library build\temp.win-amd64-3.6\Release\src/ext\fastbinary.cp36-win_amd64.lib and object build\temp.win-amd64-3.6\Release\src/ext\fastbinary.cp36-win_amd64.exp
  Generating code
  Finished generating code
  Building Custom Rule C:/projects/thrift/lib/py/CMakeLists.txt
Command exited with code 1


MoneyFieldWidget

In demo (didn't try anywhere else) the MoneyField is rendered empty on a change form, due to value rendered with currency in number field.
class MoneyFieldWidget(forms.widgets.NumberInput):
    def render(self, name, value, attrs=None):
        input_field = super(MoneyFieldWidget, self).render(name, value, attrs)
        return format_html('{} {}', input_field, self.currency_code)

input field is returning:
<input id="id_unit_price" name="unit_price" step="0.001" style="width: 75px; text-align: right" type="number" value="385.00 Kč" />

Hmm. The currency should be appended after the input field. Strange that
it's added inside the field. Does this also happen if you use Euro?
This is how I deliver DjangoSHOP to my customers.

Yes, EUR does the same. I will try to figure it out. For now I can
provide only basic information:
Python 3.5.1 and this release of shop.
-e
git+https://github.com/awesto/django-shop@4952f8f8884f618f56e82e1d0d1a23cc1a377613#egg=django_shop
On 2.9.2016 23:42, Jacob Rief wrote:

Hmm. The currency should be appended after the input field. Strange that
it's added inside the field. Does this also happen if you use Euro?
This is how I deliver DjangoSHOP to my customers.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/awesto/django-shop/issues/401#issuecomment-244494262,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApuajT0ny68tp9_2IVZJ_PRft93KmWDks5qmJhMgaJpZM4JzuVZ.

Problem is, that
value returns MoneyInCZK('380.000'), which
in case '%s % value prints  '380.00 Kč' (that is also the behavior of django Input)
Possible solution could be:
def render(self, name, value, attrs=None):
        input_field = super(MoneyFieldWidget, self).render(name, value.as_decimal(), attrs)
        return format_html('{} {}', input_field, self.currency_code)


unixPB: improves Ant-Contrib role compatibility

It helps improve Ant-Contrib role by replacing wget with get_url module. Also when extracing, on macos the unarchive module might fails as it asks for gtar but default macos tar is bsdtar. The workaround is to run it as command on macos machines.
Signed-off-by<EMAIL_ADDRESS>
Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

@mahdipub linter failures

Given the comment above the line, This needs to be tested on Solaris before it can be approved.

@mahdipub linter failures

Thanks, that's resolved.

Tagging in @gdams for a review since he put in the use of wget for Solaris as part of https://github.com/adoptium/infrastructure/pull/1585 but maybe the underlying problem was resolved in more recent versions of ansible


Input data files should not be modified by the process; outputs should be clearly distinct

(not urgent - but something for us to think about as we polish this code into the shape we want it to be)
I think its a bad idea that we modify our input data source files; we should aim to keep a clear distinction what is input and what is output.  Even if the layers in the geopackages aren't modified, we shouldn't add outputs to the input.  Doing this makes it unclear by looking at files if things have been processed.
A better approach might be to create a time stamped output geopackage, containing only the newly generated output layers.
A naming convention could use a suffix like 'output-yyyy-mm-dd', like:
bangkok_th_2019_1600m_buffer_output-2020-02-28.gpkg
This would allow us to relate our outputs back directly to the date they were first generated (as file system date modified dates aren't always reliable, or standard across systems).  So, when we change our code for the in earnest outputs, we can tell at a glance when an output was generated so we can't get them mixed up.
Technically, the csv could be table in the output geopackage if we wanted to keep output resources contained in a single file (since a gpkg file is just an sqlite database); although that would just be a nicety to keep things tidy.

agree! will do


Tre upgrade to v0.12.0

Merge changes from City of Helsinki up until v0.12.0 release
Change the color of cards on the front page
Change title in About Page to h2
Modify the feedback URL in the reservation confirmation page
Update tests

I already tested the fix for the blank IE page and it worked.


Added missing label 'Textual format' from format_2330

Whilst reviewing this ontology I noticed that a plain text format was missing from the "data format specification" hierarchy. Upon further inspection, format_2330 imported from the EDAM ontology is "Textual format", and is just missing the label.
My commit adds the label from EDAM to make it consistent with the other data formats that have been imported.

Thank you very much for submitting the change! I'll take a look at it tonight and get it all sorted. Please let me know if there's anything else you have questions about...

Thanks very much for this - I appreciate the update. This has led me down a separate path which might involve some reorganization of the format hierarchy generally, which I will ponder for a while at #47 .
Thanks again!


[FormRecognizer] Figure out a way of testing Neural build mode

We currently have a StartBuildModelWithNeuralBuildMode test, but it can take too long to run (up to 1 hour), and it seems to be related with multiple failures in our live test pipeline. We are currently ignoring the test if running live.
We should consider re-enabling the test or figuring out some other way of testing this build mode live.

There's no particular need to test neural build mode in Live test mode. We already have it enabled for Record/Playback and this has worked well in the last year. Closing this issue.
PR https://github.com/Azure/azure-sdk-for-net/pull/37618 removes the TODO message in code referencing this issue.


Deleting a class does not delete any associated relationships

When I create two classes, A and B, create a relationship between them, then delete class B, the relationship between A and B still exists 😬

On a class deletion all relationships with the deleted class are now also deleted.

Closed with PR #78.